Skip to content
Snippets Groups Projects

Archery woes (review?)

Closed Alex Ksandra requested to merge 0player/t-engine4:archery_woes into master

Since Shoot talent does not account for the run-out ammo in its pre_usee and just returns nil from action in that case (I believe that is intended for either player misuse or difficult scenarios), and as error message of not having one of two things equipped in exactly the right fashion have always been the same confusiong snippet through all archery talents, I've decided to unify the pre use check for all archery. It comes in two fashions, counting wardens in and not counting wardens in; if desirable, those could be joined for added adventurer benefit, but I just left things as they are. Playtesting suggests it indeed is working as intended now. If there's anything to add to the picture while I'm at it, or if there are any problems, please comment.

Merge request reports

Approval is optional

Closed by avatar (Apr 7, 2025 11:47pm UTC)

Merge details

  • The changes were not merged into master.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading