-
-
Notifications
You must be signed in to change notification settings - Fork 189
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
Technical: Fully move DungeonAPI.dungeonFloor to enum #1859
Technical: Fully move DungeonAPI.dungeonFloor to enum #1859
Conversation
src/main/java/at/hannibal2/skyhanni/features/dungeon/DungeonShadowAssassinNotification.kt
Outdated
Show resolved
Hide resolved
src/main/java/at/hannibal2/skyhanni/features/misc/discordrpc/DiscordLocationKey.kt
Outdated
Show resolved
Hide resolved
src/main/java/at/hannibal2/skyhanni/features/misc/discordrpc/DiscordStatus.kt
Outdated
Show resolved
Hide resolved
DungeonFloor.E -> "The Watcher" | ||
DungeonFloor.F1, DungeonFloor.M1 -> "Bonzo" | ||
DungeonFloor.F2, DungeonFloor.M2 -> "Scarf" | ||
DungeonFloor.F3, DungeonFloor.M3 -> "The Professor" | ||
DungeonFloor.F4, DungeonFloor.M4 -> "Thorn" | ||
DungeonFloor.F5, DungeonFloor.M5 -> "Livid" | ||
DungeonFloor.F6, DungeonFloor.M6 -> "Sadan" | ||
DungeonFloor.F7, DungeonFloor.M7 -> "Maxor" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DungeonFloor.E -> "The Watcher" | |
DungeonFloor.F1, DungeonFloor.M1 -> "Bonzo" | |
DungeonFloor.F2, DungeonFloor.M2 -> "Scarf" | |
DungeonFloor.F3, DungeonFloor.M3 -> "The Professor" | |
DungeonFloor.F4, DungeonFloor.M4 -> "Thorn" | |
DungeonFloor.F5, DungeonFloor.M5 -> "Livid" | |
DungeonFloor.F6, DungeonFloor.M6 -> "Sadan" | |
DungeonFloor.F7, DungeonFloor.M7 -> "Maxor" | |
DungeonFloor.F7, DungeonFloor.M7 -> "Maxor" | |
else -> floor.bossName |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i'm confused about this one
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just simplifying the code. It's different for F7/M7 because it checks for the first boss you fight (Maxor) instead of the final boss (Necron).
This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes. |
…as-enum # Conflicts: # src/main/java/at/hannibal2/skyhanni/features/misc/ParticleHider.kt
Conflicts have been resolved! 🎉 |
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/features/dungeon/DungeonAPI.kt
Conflicts have been resolved! 🎉 |
Dependencies
What
Describe what this pull request does, including technical details, screenshots, links to discord, etc.
Images
Changelog New Features
Changelog Improvements
Changelog Fixes
Changelog Technical Details
Changelog Removed Features