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

Projectile ammo matches ItemAction instead of age #2453

Merged
merged 3 commits into from
Feb 15, 2023

Conversation

Cardboy777
Copy link
Contributor

@Cardboy777 Cardboy777 commented Feb 11, 2023

*Part of an old PR I posted months ago. Figured a smaller PR per each 'feature' would be better for code review.

Makes slingshot shoot seeds, and bow shoot arrows for both ages. Made sure that the shooting gallery minigames gave the player the correct item for their age.

I don't know what the general policy about this kind of change is; whether it would be better to have it behind an enhancement/fix toggle or not.

Build Artifacts

extracted logic from months-old PR
Copy link
Contributor

@briaguya-ai briaguya-ai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is changing authentic behavior and needs to be behind a cvar

@Cardboy777
Copy link
Contributor Author

this is changing authentic behavior and needs to be behind a cvar

Done. I had trouble choosing the correct submenu to put it under. I ended up putting it in "Enhancements -> Gameplay -> Items", but was also considering "Enhancements -> Fixes"

@briaguya-ai briaguya-ai merged commit 218971c into HarbourMasters:develop Feb 15, 2023
@Cardboy777 Cardboy777 deleted the ProjectileAmmoFix branch February 15, 2023 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants