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

Shaded Hills Beggar Knight changes #4225

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

Greenjoe12345
Copy link
Contributor

Description of changes

Adds different alt titles for beggar knight, allowing them to either spawn with a shortsword and shield, or with a longsword. Also makes it so beggar knights do not spawn with money even if you select to have money in character setup.

Why and what will this PR improve

Being able to pick different kinds of weapon to carry is neat, and keeping knights from breaking their vow before even spawning is good.

Authorship

Myself

Changelog

🆑
add: Alt titles to allow Beggar Knights on Shaded Hills to spawn either with a shortsword and shield or a longsword.
tweak: Made it so Beggar Knights on Shaded Hills do not spawn with money.
/:cl:

@MistakeNot4892
Copy link
Contributor

Let me think about a better way to handle the equipment selection. Making the main title like that bothers me a lot.

@MistakeNot4892 MistakeNot4892 added the do not merge This PR is on hold, do not merge it. label Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge This PR is on hold, do not merge it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants