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: Hypixel Mod API #2768

Merged
merged 17 commits into from
Jan 8, 2025
Merged

Conversation

ItsEmpa
Copy link
Contributor

@ItsEmpa ItsEmpa commented Oct 19, 2024

What

This PR implements the Hypixel Mod API and creates HypixelAPIServerChangeEvent with it.
The goal of using the mod api is having more precise information, like hypixel, skyblock, and island detection. This pr just adds the functionality but doesnt use it: it only compares it to the one in the current system, and sends an error message in chat if the comparison fails. This is gonna be like this for a couple betas, and if things go well the old system will start getting changed to use the mod api.

Changelog Technical Details

  • Implemented Hypixel Mod API. - Empa
  • Added HypixelAPIServerChangeEvent. - Empa

@github-actions github-actions bot added Backend A backend pull request that will be merged soon Wrong Title/Changelog There is an error in the title or changelog labels Oct 19, 2024
Copy link

I have detected some issues with your pull request:

Body issues:
Change should end with a full stop in text: Added HypixelAPIServerChangeEvent,

Please fix these issues. For the correct format, refer to the pull request template.

@github-actions github-actions bot removed the Wrong Title/Changelog There is an error in the title or changelog label Oct 19, 2024
@hannibal002 hannibal002 added this to the Version 0.29 milestone Oct 19, 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 added the Merge Conflicts There are open merge conflicts with the beta branch. label Nov 28, 2024
# Conflicts:
#	src/main/java/at/hannibal2/skyhanni/config/features/dev/DevConfig.java
#	src/main/java/at/hannibal2/skyhanni/data/HypixelData.kt
#	src/main/java/at/hannibal2/skyhanni/events/HypixelJoinEvent.kt
@github-actions github-actions bot removed the Merge Conflicts There are open merge conflicts with the beta branch. label Jan 5, 2025
Copy link

github-actions bot commented Jan 5, 2025

Conflicts have been resolved! 🎉

Signed-off-by: Empa <[email protected]>
@jani270 jani270 added the Soon This Pull Request will be merged within the next couple of betas label Jan 5, 2025
@github-actions github-actions bot added the Detekt Has detekt problem label Jan 5, 2025
Copy link

github-actions bot commented Jan 5, 2025

One or more Detekt Failures were detected:

@github-actions github-actions bot removed the Detekt Has detekt problem label Jan 5, 2025
@hannibal002 hannibal002 merged commit b50fa05 into hannibal002:beta Jan 8, 2025
6 checks passed
@github-actions github-actions bot removed Soon This Pull Request will be merged within the next couple of betas Backend A backend pull request that will be merged soon labels Jan 8, 2025
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.

4 participants