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

Feature + Fix: Hoppity Event Summarization #2311

Merged
merged 43 commits into from
Aug 26, 2024

Conversation

DavidArthurCole
Copy link
Contributor

@DavidArthurCole DavidArthurCole commented Aug 5, 2024

What

https://discord.com/channels/997079228510117908/1262558938650378250
Adds a recap for Hoppity Events, added an event that gets fired when a rabbit is collected. Reworked the HoppityEggType enum. Tangentially related to the enum, this PR also adds functionality for Hoppity Compact to compact:

  • Side Dish rabbits
  • All-Time Chocolate Milestone rabbits
  • Shop milestone rabbits
Images

image

image

image

Changelog New Features

  • Added Hoppity Hunt event summary. - Daveed
    • Use /shhoppitystats for live stats.

Changelog Fixes

  • Fixed "Side Dish" and Milestone rabbits not sending a message when Compact Hoppity is enabled. - Daveed

Changelog Technical Details

  • Added HoppityAPI, which should house event-specific features rather than ChocolateFactoryAPI. - Daveed
    • Added an Always Hoppity dev config option, which will override isHoppityEvent.
  • Added RabbitFoundEvent, which will be fired when a chocolate rabbit is found from an egg, bought from Hoppity, or claimed from a milestone. - Daveed
  • Added SIDE_DISH, CHOCOLATE_SHOP_MILESTONE, CHOCOLATE_FACTORY_MILESTONE, and BOUGHT as HoppityEggType options. - Daveed

@hannibal002 hannibal002 added this to the Version 0.27 milestone Aug 8, 2024
@hannibal002 hannibal002 changed the title Feature: Hoppity Event Summarization Feature + Fix: Hoppity Event Summarization Aug 13, 2024
@github-actions github-actions bot added the Bug Fix Bug fixes label Aug 13, 2024
@DavidArthurCole
Copy link
Contributor Author

Couple bugs I need to fix still:

  • Leaderboard change needs to be removed, it's needlessly complicating the entire module
  • Found eggs divisor is 0 for historical data, should default to 279 for past data

Copy link

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Aug 16, 2024
@hannibal002 hannibal002 added the Waiting on Hypixel Stuff that cant be tested right now label Aug 23, 2024
Copy link

This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes.

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Aug 25, 2024
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Aug 25, 2024
Copy link

Conflicts have been resolved! 🎉

Copy link
Owner

@hannibal002 hannibal002 left a comment

Choose a reason for hiding this comment

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

works fine so far, only small things

@hannibal002 hannibal002 merged commit 290676b into hannibal002:beta Aug 26, 2024
4 checks passed
@github-actions github-actions bot removed Waiting on Hypixel Stuff that cant be tested right now Bug Fix Bug fixes labels Aug 26, 2024
@DavidArthurCole DavidArthurCole deleted the HoppitySummary branch August 26, 2024 00:43
MTOnline69 pushed a commit to MTOnline69/SkyHanni that referenced this pull request Sep 11, 2024
Co-authored-by: hannibal2 <24389977+hannibal00212@users.noreply.github.com>
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.

2 participants