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

Add missing item mappings #681

Closed
wants to merge 2 commits into from

Conversation

TheLadyLuthien
Copy link
Contributor

Fix many mappings for items in Items that were missing.

@TheLadyLuthien
Copy link
Contributor Author

Oddly enough, after looking into it, it seems to have only broken between 1.21.1-build.3 and 1.21.1-build.4. In 1.21.1.3 they seem to be auto-mapped by the enigma extension, but in 1.21.1.4 they are unmapped.

dead brain, more like brain dead

Co-authored-by: Will <[email protected]>
@ix0rai
Copy link
Member

ix0rai commented Jan 5, 2025

really sorry i didn't catch earlier that you wanted to work on this, it's an issue that iota and i previously encountered and fixed in the backend. since enigma/plugin versions are usually only updated on the latest branch, the fixes did not come back to 1.21.1. as we don't like putting in redundant mappings, i'll be superceding this PR with an enigma/plugin update. again, i'm really sorry i didn't get a chance to tell you we had already fixed our automapping!

@ix0rai ix0rai added t: new adds new mappings v: release targets a release version of minecraft s: small PRs with less than 200 lines labels Jan 5, 2025
@supersaiyansubtlety
Copy link
Member

superseded by #682

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s: small PRs with less than 200 lines t: new adds new mappings v: release targets a release version of minecraft
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants