Skip to content

Conversation

@Chr1Z93
Copy link
Collaborator

@Chr1Z93 Chr1Z93 commented Jul 26, 2024

  • various bugfixes relating to empty / non-matching "uses"
  • updated feedback message
  • also updated feedback message logic for "Take a clue"
  • fixed a bug for spawning a resource on cards without "uses"

@Chr1Z93 Chr1Z93 added the bugfix fixing an existing bug label Jul 26, 2024
@Chr1Z93 Chr1Z93 linked an issue Jul 26, 2024 that may be closed by this pull request
Copy link
Collaborator

@dscarpac dscarpac left a comment

Choose a reason for hiding this comment

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

The wording on the new message is a bit awkward. "Removed a Charge" is OK, but "Removed an Ammo" isn't quite right in English, it'll need to be "an Ammo token" or something similar. Not sure what was wrong with the old way, which is a little less awkward sounding between the game language and English.

Another one: the name of the resource counter is "Resourcecounter" without a space.

@Chr1Z93
Copy link
Collaborator Author

Chr1Z93 commented Jul 27, 2024

The wording on the new message is a bit awkward. "Removed a Charge" is OK, but "Removed an Ammo" isn't quite right in English, it'll need to be "an Ammo token" or something similar. Not sure what was wrong with the old way, which is a little less awkward sounding between the game language and English.

Another one: the name of the resource counter is "Resourcecounter" without a space.

There was nothing wrong with it but it didn't allow adding the location easily.
I've updated it.

@dscarpac dscarpac merged commit ecfab0d into main Jul 28, 2024
@dscarpac dscarpac deleted the removeuse branch July 28, 2024 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix fixing an existing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix: Token spawning / removal

3 participants