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

Add Error Messages to the Downgrade Ritual #1990

Open
wants to merge 3 commits into
base: 1.20.1
Choose a base branch
from

Conversation

VT-14
Copy link
Collaborator

@VT-14 VT-14 commented Oct 10, 2023

This gives clear indications to the player about what is going wrong for the most commonly seen issues.

  • A Training Bracelet is blocking the Downgrade.
  • The player doesn't have enough of the Key Item (message appears if what they already have is >= what is in the chest, most likely they have level 1 and only 1 Key item in the chest).
  • The player doesn't have enough Upgrade Points in the chest to apply the Downgrades they are requesting.
  • The inventory to hold the downgrade recipe is missing.

This gives clear indications to the player about what is going wrong for the most commonly seen issues.

* A Training Bracelet is blocking the Downgrade.
* The player doesn't have enough of the Key Item (message appears if what they already have is >= what is in the chest, most likely they have level 1 and only 1 Key item in the chest).
* The player doesn't have enough Upgrade Points in the chest to apply the Downgrades they are requesting.
@VT-14 VT-14 added the 1.20 label Oct 10, 2023
VT-14 added 2 commits October 10, 2023 11:16
Changed the notEnoughKeyItems message to display the item's default name rather than the generic "Key Item".  Minor grammar tweak to make it work.

Changed the Training Bracelet's message from "shakes" to "vibrates."
Add an error message for the Downgrade Ritual for when the inventory is missing.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant