-
-
Notifications
You must be signed in to change notification settings - Fork 213
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
Improvement: Open Last Storage now saves across instances #3182
base: beta
Are you sure you want to change the base?
Conversation
One or more Detekt Failures were detected:
|
src/main/java/at/hannibal2/skyhanni/features/commands/OpenLastStorage.kt
Outdated
Show resolved
Hide resolved
Should be ready for review, will test more tomorrow though (at least I hope so as Physics exam on friday 😭) |
I have detected some issues with your pull request: Body issues: Please fix these issues. For the correct format, refer to the pull request template. |
src/main/java/at/hannibal2/skyhanni/config/features/misc/MiscConfig.java
Show resolved
Hide resolved
This pull request has conflicts with the base branch "beta". Please resolve those so we can test out your changes. |
Conflicts have been resolved! 🎉 |
Ready to merge, tested and works really good |
src/main/java/at/hannibal2/skyhanni/config/features/misc/LastStorageConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/at/hannibal2/skyhanni/config/storage/ProfileSpecificStorage.kt
Outdated
Show resolved
Hide resolved
src/main/java/at/hannibal2/skyhanni/features/commands/OpenLastStorage.kt
Outdated
Show resolved
Hide resolved
src/main/java/at/hannibal2/skyhanni/config/storage/ProfileSpecificStorage.kt
Outdated
Show resolved
Hide resolved
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.
code lgtm
One or more Detekt Failures were detected:
|
What
Improves Open Last Storage to now save across instances
Changelog Improvements