jaquadro / StorageDrawers

A mod adding compartmental storage for Minecraft Forge
MIT License
203 stars 160 forks source link

Can the fabric version not depend on the Forge Config API Port? #1160

Closed swe3per closed 1 month ago

swe3per commented 1 month ago

The compatibility of Forge Config API Port in higher versions is not good enough, which causes the server to be unable to shut down normally. Can subsequent versions consider not relying on this module? Such a powerful and easy-to-use drawer mod finally supports fabric. I don’t want the quality to be affected by the dependent mods. :)

jaquadro commented 1 month ago

It can probably be made optional with the obvious caveat that you will not get any config support available for the mod. In the interim you can of course step back one version, as the only change was the reintroduction of config support

swe3per commented 1 month ago

This is a good idea. If it is installed on the server, is it also impossible to modify the configuration by modifying the toml file?

jaquadro commented 1 month ago

If there's no forge config API port present, then there are no toml files to configure. The mod loads static defaults internally. Forge config API port seemed like the obvious solution to carry over config support but if there's other solutions in common use that aren't too hard to adapt I could look into them.

swe3per commented 1 month ago

Thanks for your help. I'm sorry that I can only communicate with you from the perspective of a player/OP. When choosing a MOD, we often feel anxious because of powerful but unstable dependent MOD. I can understand the convenience these dependencies provide to developers, so I put forward this immature suggestion. Thank you very much for your understanding.

jaquadro commented 1 month ago

I published 13.7.2, which should make the dependency optional. No ETA on when configuration might be available by a different method.

swe3per commented 1 month ago

Thanks a lot my bro!