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: Add SkyHanni User Luck to stats breakdown #1288

Merged
merged 20 commits into from
Jun 16, 2024

Conversation

martimavocado
Copy link
Contributor

@martimavocado martimavocado commented Mar 27, 2024

What

Adds the SkyHanni User Luck stat to the stats breakdown in /sbmenu, /equipment and "Misc Stats" in it's submenu.
Follows the same visibility rules as other Hypixel stats. Example: Only visible when on a non default value, respects "Toggle show all stats".
Adds a custom stats breakdown menu for SkyHanni User Luck, has support for future new sources, but would probably need a bit of extra work for a new sub-sub menu for a future skills breakdown.

very much needs a new stat description, maybe a new item to represent the general stat.

video showcase
screenshots are a bit outdated and don't include the new menu
image
image
image

Changelog New Features

  • Added SkyHanni User Luck to the stats breakdown. - martimavocado
    • This can be viewed in /sbmenu, /equipment, and the Misc Stats submenu.

@hannibal002 hannibal002 added this to the Version 0.25 milestone Mar 27, 2024
Copy link

github-actions bot commented Apr 3, 2024

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 Apr 3, 2024
Copy link

github-actions bot commented Apr 3, 2024

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

# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/SkyHanniMod.kt
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Apr 10, 2024
Copy link

Conflicts have been resolved! 🎉

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 Apr 13, 2024
# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/SkyHanniMod.kt
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Apr 19, 2024
@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label May 24, 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 removed the Merge Conflicts There are open merge conflicts with the beta branch. label May 24, 2024
Copy link

Conflicts have been resolved! 🎉

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

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

# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/SkyHanniMod.kt
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Jun 9, 2024
Copy link

github-actions bot commented Jun 9, 2024

Conflicts have been resolved! 🎉

@hannibal002 hannibal002 added the Soon This Pull Request will be merged within the next couple of betas label Jun 15, 2024
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.

missing addSeparators()

Images

grafik
grafik

@hannibal002 hannibal002 merged commit dadf6c0 into hannibal002:beta Jun 16, 2024
4 checks passed
@github-actions github-actions bot removed the Soon This Pull Request will be merged within the next couple of betas label Jun 16, 2024
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.

3 participants