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

Upgrades Custom Messages to Hooks and ShipInit #5101

Draft
wants to merge 46 commits into
base: develop
Choose a base branch
from

Conversation

leggettc18
Copy link
Contributor

@leggettc18 leggettc18 commented Feb 28, 2025

This PR is my effort to convert our old Custom Message systems to use the OnTextLoad hook that I backported from 2Ship, and ShipInit to replace our previous methods for displaying Custom Messages. Some articles, colors, and icons got lost in translation that I know of, so it's not 100% done yet. However these are all visual, so playtesters are safe to go ahead and try this build out to see if they can find any bugs like text popping up in the wrong places or hooks not getting registered properly.

Build Artifacts

Only for the items, other things previously covered by CustomMessage_Retrieve if exists have not yet been covered as of this commit and will either load their vanilla text or crash.
Specifically Dampe, Greg, Warp Songs, Frogs, Loach, Fishing Pole, and Saria
This is the last of the static hints
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.

1 participant