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

Fanning Should Go in 1 Direction #513

Open
albertoxamin opened this issue Aug 3, 2024 · 0 comments
Open

Fanning Should Go in 1 Direction #513

albertoxamin opened this issue Aug 3, 2024 · 0 comments

Comments

@albertoxamin
Copy link
Owner

It says fanning "Also targets the player of your choice at distance from first target (if any, excluding you) with a bang, so I think this needs to be reworked in the game. Currently you can target anyone.

albertoxamin added a commit that referenced this issue Aug 3, 2024
…st target

Related to #513

Update the `Sventagliata` card to restrict the secondary target to be at a distance from the first target.

* **backend/bang/expansions/the_valley_of_shadows/cards.py**
  - Modify the `play_card` method in the `Sventagliata` class to enforce distance constraints for the secondary target.
  - Ensure the secondary target selection logic only includes players at a distance of 1 from the first target.

* **backend/tests/valley_of_shadows_test.py**
  - Update the `test_sventagliata` test to verify the secondary target is at a distance from the first target.
  - Add assertions to ensure the secondary target is correctly chosen based on the distance constraint.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/albertoxamin/bang/issues/513?shareId=XXXX-XXXX-XXXX-XXXX).
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

No branches or pull requests

1 participant