Skip to content

chore: make nbtac conflict with nbt_ac - #70

Merged
Wyvest merged 3 commits into
Polyfrost:mainfrom
lunaynx:chore/nbtac-conflict
Sep 8, 2026
Merged

chore: make nbtac conflict with nbt_ac#70
Wyvest merged 3 commits into
Polyfrost:mainfrom
lunaynx:chore/nbtac-conflict

Conversation

@lunaynx

@lunaynx lunaynx commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Description

Mod dev is dumb and changed the mod ID between 1.x and 2.x?? I've seen at least two people crash with a @Redirect conflict due to having both installed today. This should give them a clearer error message.

Related Issue(s)

Fixes #

How to test

Release Notes

Documentation

@Wyvest

Wyvest commented Sep 4, 2026

Copy link
Copy Markdown
Member

I would merge but lowkey im not sure if oneclient launcher would merge this in properly...

@lunaynx

lunaynx commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

It seems to work (I manually edited the local file and then removed write permission so that it doesn't overwrite). Though the error isn't great for the average user, it's just more obvious for support.

I wonder if we can inject early in Poly+ and flag this or quietly prevent the older version of the mod from loading if the newer one is loaded...

@Wyvest
Wyvest merged commit 9b47864 into Polyfrost:main Sep 8, 2026
2 checks passed
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