Skip to content

Update to Minecraft 26.2#391

Merged
John-Paul-R merged 5 commits into
John-Paul-R:26.xfrom
eclipseisoffline:26.2
Jun 17, 2026
Merged

Update to Minecraft 26.2#391
John-Paul-R merged 5 commits into
John-Paul-R:26.xfrom
eclipseisoffline:26.2

Conversation

@eclipseisoffline

Copy link
Copy Markdown
Contributor

This PR updates the mod to Minecraft 26.2 (currently release candidate 2). With this update, the mod moves to use Fabric API's new permissions API, which uses Minecraft's identifiers as permission nodes.

Permission managers like LuckPerms "flatten" these identifiers into a string by replacing the : with a .. For example, essentialcommands:tpa turns into essentialcommands.tpa. As such, the user-facing permission strings when using LuckPerms should remain unchanged.

Unsure on how to address the failing tests at the moment.

@John-Paul-R

Copy link
Copy Markdown
Owner

Thanks for the upgrade! I'll review today.

Comment thread ec-core/src/main/java/dev/jpcode/eccore/config/ConfigUtil.java Outdated
@arnokeesman

Copy link
Copy Markdown
Contributor

tests also seem to run fine if I don't update gradle and loom, so guess there's some gradle hell ahead
succesful tests on a minimal update: https://github.com/arnokeesman/Essential-Commands/actions/runs/27651159484

@John-Paul-R

John-Paul-R commented Jun 17, 2026

Copy link
Copy Markdown
Owner

currently worried this is going to use the mod_id (which is essential_commands) as the permissions namespace by default (instead of essentialcommands, which it has been to this point). if it does, then this will be a breaking permissions change for all users. I may need to shim in a fix to avoid that.

nevermind, I simply failed to read. Huzzah!

    public static Identifier of(String name) {
        return Identifier.fromNamespaceAndPath("essentialcommands", name);
    }

@John-Paul-R

Copy link
Copy Markdown
Owner

Looks good to me. Thank you!

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.

3 participants