Skip to content

Prefer random token from set - #7961

Merged
kevlahnota merged 7 commits into
Card-Forge:masterfrom
Eradev:RandomToken
Jul 8, 2025
Merged

Prefer random token from set#7961
kevlahnota merged 7 commits into
Card-Forge:masterfrom
Eradev:RandomToken

Conversation

@Eradev

@Eradev Eradev commented Jul 7, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread forge-core/src/main/java/forge/token/TokenDb.java Outdated
@Hanmac

Hanmac commented Jul 7, 2025

Copy link
Copy Markdown
Contributor

i don't really see the reason behind this

And artIndex is going to be replaced by #7846 anyway soon?

@Eradev

Eradev commented Jul 7, 2025

Copy link
Copy Markdown
Contributor Author

I'm not sure what you're working on, but I just wanted to use the set's tokens when possible so it isn't always the default one. One of Magic's strength IMO is the wide different arts and being locked into one when there would be more flavorful ones is depressing.

@tool4ever tool4ever left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@Hanmac Hanmac left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

i don't really like the changes with the art index
but i'm going to remove them later anyway

@kevlahnota
kevlahnota merged commit 6526d73 into Card-Forge:master Jul 8, 2025
@Eradev
Eradev deleted the RandomToken branch July 9, 2025 09:05
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.

4 participants