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

feat: support iidx 31-2dxtra #1233

Merged
merged 6 commits into from
Jan 17, 2025
Merged

feat: support iidx 31-2dxtra #1233

merged 6 commits into from
Jan 17, 2025

Conversation

GRIM657
Copy link
Contributor

@GRIM657 GRIM657 commented Jan 11, 2025

This adds support for Epolis 2dxtra, and also shoehorns in some altTitles for Pinky Crush CSVs.

This is my first time adding a 2dxtra version, but it looks like everything went well? Thanks for making it easy :)

@tachi-repo-bot
Copy link

A commit has changed the seeds. View the diff here.

@GRIM657
Copy link
Contributor Author

GRIM657 commented Jan 11, 2025

Ah, nevermind, I should have re-started the server locally and caught these issues. I think the deprimary logic isn't working for cases where a song's 2dxtra inGameID has changed due to infinitas ports etc. I'm going to mark this WIP for now.

@GRIM657 GRIM657 changed the title feat: support iidx 31-2dxtra WIP: feat: support iidx 31-2dxtra Jan 11, 2025
@tachi-repo-bot
Copy link

A commit has changed the seeds. View the diff here.

@GRIM657 GRIM657 changed the title WIP: feat: support iidx 31-2dxtra feat: support iidx 31-2dxtra Jan 11, 2025
@tachi-repo-bot
Copy link

A commit has changed the seeds. View the diff here.

@GRIM657
Copy link
Contributor Author

GRIM657 commented Jan 11, 2025

OK we should be good now :)

Also, there were three unreleased charts in 32 that were revealed like the day after I sent in the PR lol. So I slipped those in too.

@zkrising zkrising merged commit de3fcd0 into zkrising:main Jan 17, 2025
8 checks passed
@GRIM657 GRIM657 deleted the 31-extra branch January 18, 2025 00:19
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