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

Fix Scavenger's Escort Service to allow movement of one hex #2653

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aybanda
Copy link
Contributor

@aybanda aybanda commented Nov 27, 2024

This fixes issue #2616

Description of Changes

This pull request addresses the issue with the "Scavenger's Escort Service" ability by allowing it to function correctly even when moving a target unit by only one hex. The following changes have been made:

  • Updated the require() function to allow the ability to be activated with one usable hex.
  • Refined the filtering logic to ensure the ability can still be used effectively.
  • Adjusted the movement logic to ensure at least one movement point is deducted.

Wallet Address

My wallet address is 0xfBA90c8Fd61AcD3aa1e086628dc402Aa34334aDc

Copy link

vercel bot commented Nov 27, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ancientbeast ✅ Ready (Inspect) Visit Preview Nov 27, 2024 6:52am

@DreadKnight
Copy link
Member

@aybanda Heya! Thanks for the PR. I've increased XTR bounty for this. I've tested this and works like in master, no change:

Screenshot_20241127_152842
Screenshot_20241127_152904

@DreadKnight DreadKnight marked this pull request as draft December 8, 2024 03:46
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.

2 participants