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

chore: Update to v2 NEX libraries #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ItzSwirlz
Copy link

@ItzSwirlz ItzSwirlz commented Aug 11, 2024

Resolves #2

Changes:

  • Updated to nex-go v2 and equivalent NEX libraries

  • Uses common ranking protocol

  • Renamed method and file GetRankingsByCategoryAndRankingOrderParam to proper full name GetRankingsAndCountByCategoryAndRankingOrderParam

  • Updated database functions to work with the v2 ranking protocol

  • I have read and agreed to the Code of Conduct.

  • I have read and complied with the contributing guidelines.

  • What I'm implementing was an approved issue.

  • I have tested all of my changes.

@ItzSwirlz ItzSwirlz marked this pull request as draft August 11, 2024 14:19
@ItzSwirlz
Copy link
Author

I have confirmed this works just as much as it did on the current server where you can upload mission scores, but BOSS ranking doesn't work yet. I'm grabbing a 100% save to test the rest of the functionality.

@ItzSwirlz
Copy link
Author

Can't seem to figure it out: besides the game rankings are done via BOSS. The main purpose of the server is just to upload the scores in missions to my knowledge, so it works.

@ItzSwirlz ItzSwirlz marked this pull request as ready for review August 11, 2024 14:59
@ItzSwirlz ItzSwirlz changed the title [WIP] chore: Update to v2 NEX libraries chore: Update to v2 NEX libraries Aug 11, 2024
@ashquarky
Copy link
Member

Code looks fine to my eyes, though I don't have this game to test right now

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.

[Enhancement]: Update to v2 NEX libraries
2 participants