-
-
Notifications
You must be signed in to change notification settings - Fork 226
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
Fixes & Improvements: Hoppity Rework #2488
Fixes & Improvements: Hoppity Rework #2488
Conversation
So I wasn't actually accounting for waypoint sharing in the "every hoppity message consists of 3" - but that's fixed now, so... |
Nvm now purchased rabbits broke, which is dope considering I didn't touch those at all 🔥 |
"Compact Hoppity"
Delay
I'm realizing this PR's scope creeped rather massively as I found more and more small issues with Hoppity's, but uhhh yeah there are now fixes & improvements. Still testing a couple things out before I'm 100% confident on some of the assumptions made - need next hoppity to come around to verify that event firing won't overlap. |
What
This PR involved an almost total rework of the chat detection that is inherent to the Hoppity "module":
Removes the 200ms delay that was added to Hoppity Compact in the early days when the number of messages was indeterminate (or, not always exactly 3).
See details and images below, but all "types" of rabbits found now consist of exactly three messages to spin them off, so the delay is no longer needed.
* The one exception to the "three-message rule" is when waypoint sharing is enabled, however this is still being accounted for separately and within the module itself.
This PR also addresses the "issue" that had been raised previously by hannibal (see here) about using
LorenzChatEvent()
to fake messages for:Instead of doing that, it's using an event registry now for
EggFoundEvent()
, and leaving it up to the modules to determine what to do with that. This was most prevalent for the Compact Hoppity stuff.Images
Meal (Normal):

Bought:

Side Dish+ (the same logic here applies to Factory and Shop milestones, since they all use a manually-created chat event for type declaration):

Strays (again, contrived third message, but - also the

messageCount
can be ignored):Changelog Fixes
Changelog Improvements
Changelog Technical Details
LorenzChatEvent()
for 'manually' adding data to Hoppity Chat Compact. - DaveedEggFoundEvent()
.