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

Backend: MaxwellAPI #2850

Open
wants to merge 6 commits into
base: beta
Choose a base branch
from

Conversation

ItsEmpa
Copy link
Contributor

@ItsEmpa ItsEmpa commented Oct 28, 2024

Dependencies

What

This PR refactors the MaxwellAPI to make it use MaxwellPower data classes instead of just storing the name, to allow for adding more data to powers in the future. Also adds Update events for power, mp and tunings, which will be used for custom scoreboard (soon™️).

Changelog Technical Details

  • Refactored MaxwellAPI to use MaxwellPower data classes instead of strings. - Empa

@github-actions github-actions bot added the Backend A backend pull request that will be merged soon label Oct 28, 2024
Copy link

One or more Detekt Failures were detected:

@hannibal002 hannibal002 added this to the Version 0.29 milestone Oct 28, 2024
@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Nov 15, 2024
Copy link

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

…rework

# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/data/maxwell/MaxwellAPI.kt
#	src/main/java/at/hannibal2/skyhanni/features/inventory/StatsTuning.kt
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Nov 15, 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 Nov 15, 2024
# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/features/inventory/StatsTuning.kt
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Nov 25, 2024
Copy link

Conflicts have been resolved! 🎉

@github-actions github-actions bot added the Detekt Has detekt problem label Nov 25, 2024
Copy link

One or more Detekt Failures were detected:

@github-actions github-actions bot added the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 4, 2025
Copy link

github-actions bot commented Jan 4, 2025

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

@hannibal002 hannibal002 modified the milestones: 2.0.0, 3.0.0, 4.0.0 Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend A backend pull request that will be merged soon Detekt Has detekt problem Merge Conflicts There are open merge conflicts with the beta branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants