-
-
Notifications
You must be signed in to change notification settings - Fork 216
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
Backend: PetAPI V2 #3042
base: beta
Are you sure you want to change the base?
Backend: PetAPI V2 #3042
Conversation
# Conflicts: # src/main/java/at/hannibal2/skyhanni/config/commands/Commands.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 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/inventory/experimentationtable/ExperimentationTableAPI.kt # src/main/java/at/hannibal2/skyhanni/features/misc/pets/CurrentPetDisplay.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 have been 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! 🎉 |
What
Continuation PR of #2605, reforming the PetAPI in preparations for overhauling the active pet display. This does a little more work on top of what @martimavocado had done, to clean up the way data is being populated - as such, this PR is based on his branch.
What was previously
PetAPI
has now been split into:PetData
CurrentPetAPI
PetUtils
Changelog Technical Details
CurrentPetAPI
,PetUtils
, andPetData
.