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

Hidden and Double Agenda better as Keyword #7093

Merged
merged 2 commits into from
Feb 28, 2025
Merged

Conversation

Hanmac
Copy link
Contributor

@Hanmac Hanmac commented Feb 27, 2025

this is for reducing the use of hasKeyword("string")

i couldn't test it really, because my draft isn't working as i want

@tool4ever
Copy link
Contributor

looks ok I guess?

@Hanmac Hanmac marked this pull request as ready for review February 27, 2025 08:21
@tool4ever
Copy link
Contributor

@Hanmac
I think for testing you can also manually add them in deck editor?

@Hanmac
Copy link
Contributor Author

Hanmac commented Feb 27, 2025

@tool4ever yeah i tried with draft but that didn't work
i didn't know if could add them via deck editor. (or somehow cheat them?)

can you test them until can check on them later?

@Jetz72
Copy link
Contributor

Jetz72 commented Feb 27, 2025

I believe you can add them to a constructed deck in the mobile editor at least, but no variant besides draft will actually utilize them in game. You could probably test it by opening up RegisteredPlayer.java, go down to the restoreDeck() method and add assignConspiracies(); to the bottom of it.

@Hanmac Hanmac requested a review from tool4ever February 28, 2025 08:16
@Hanmac Hanmac force-pushed the hiddenAgendaRework branch from 2ad9414 to 5ba7728 Compare February 28, 2025 09:03
@Hanmac Hanmac merged commit d02dd67 into master Feb 28, 2025
6 checks passed
@Hanmac Hanmac deleted the hiddenAgendaRework branch February 28, 2025 09:22
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.

3 participants