Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Makes a particular Keep door ridethrough (for mounts) #1096

Conversation

FreeStylaLT
Copy link

About The Pull Request

dreamseeker_U2fMRXO2K3
this one

Why It's Good For The Game

angel-blob-emoji-blob-emoji-3425341223

@burgerenergy
Copy link

ideally there'd be something to highlight this secret interaction to players

i have no ideas as to what, though.

although the gate is right there...

@FreeStylaLT
Copy link
Author

FreeStylaLT commented Dec 25, 2024

ideally there'd be something to highlight this secret interaction to players

i have no ideas as to what, though.

although the gate is right there...

I'd add it to its examine text, but objects /generally/ don't have descriptions on this codebase, so you wouldn't expect anything useful to be on examine, so you wouldn't really examine it.

@radarseas
Copy link

they already have the gate for this, if the keep wants to lead out mounted warriors, they should be using the gate controls, which aren't that far off from this door anyway. Not riding through that door.

@YakumoChen
Copy link

YakumoChen commented Dec 26, 2024

Almost nobody ever mans the gate

It's a boring job.

@FreeStylaLT
Copy link
Author

ideally there'd be something to highlight this secret interaction to players

i have no ideas as to what, though.

although the gate is right there...

image

@YakumoChen
Copy link

YakumoChen commented Dec 27, 2024

Saigas are not horses

It should probably just be named an arched keep entrance to signify there is additional vertical space above the door for taller entry (with a custom description to make it clear it allows for mounted entry)

@Lutowski Lutowski merged commit ad32e60 into GeneralPantsuIsBadAtCoding:main Jan 2, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants