Flemmli97 / Flan

Fabric Landclaiming Mod
80 stars 34 forks source link

[Feature Request]: Making BuiltinRegistries.register public #284

Closed Emafire003 closed 3 months ago

Emafire003 commented 4 months ago

Suggestions

I would suggest making the registering methods for new permissions public so other mods depending on Flan can simply use those to add new permissions instead of using their own datagen and registering.

Solution

No response

Flemmli97 commented 3 months ago

you dont register your permissions via code anymore. you create a datapack and that registers them. BuiltinRegistries.register is purely for this mods datagen calling that will do nothing unless you use the datagen from this mod. and then you would also generate all permissions from this mod making lot of redundant things.

you can extend ClaimPermissionProvider to make a custom datagenerator

Emafire003 commented 3 months ago

Yeah sorry I misinterpreted the thing, and meant to close this issue already but forgot!