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

refactor: Misc fixes for API model updates #125

Merged
merged 3 commits into from
Apr 10, 2024

Conversation

Panzerhandschuh
Copy link
Member

  • Use UserMapData for checking if the map file exists or is in favorites
  • Lookup main track leaderboard since it no longer exists on the model
  • Updated map data to be compatible with new backend models

@tsa96
Copy link
Member

tsa96 commented Apr 6, 2024

JS looks fine besides not having autoformatting applied. CI should be yelling at you but the job is only configured to run on one branch.

My last PR enables CI everywhere, then #127 brings the 0.10 branch up to date with that, please could you approve and merge that, then rebase this branch onto top of latest 0.10. Then run npm run format:fix and amend those changes into this commit, then force-push.

I'll double check this when more awake tomorrow and have reviewed the red side of this stuff

@tsa96 tsa96 force-pushed the feat/misc-map-model-fixes branch from 7db0c1d to 65fba01 Compare April 6, 2024 21:36
scripts/common/map-cache.js Outdated Show resolved Hide resolved
scripts/pages/map-selector/map-entry.js Outdated Show resolved Hide resolved
@Panzerhandschuh Panzerhandschuh merged commit 00de1bd into feat/mom-0.10 Apr 10, 2024
1 check passed
@Panzerhandschuh Panzerhandschuh deleted the feat/misc-map-model-fixes branch April 10, 2024 20:45
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.

2 participants