Odd Essentials modifications about theotherroles HOT 6 CLOSED

I totally get your point and I'm on your side with that. I had the issue with people being unable to join lobbies because sharing the settings somehow didn't want to work when using string ids. Sadly I never got to the bottom of it, you probably did in the mean time? As my changes were not practical and as you're saying yourself... odd... there was no sense in opening a PR. You're right I should have opened an issue or feature request, sorry for that. You're also right with the lobby text patch, I just entered some dirty spacings/some tabs and colors in there (some of the features you already added to Essentials). I changed some accessibility levels because I needed access in order to add some settings presets you can switch through (probably not something that would make sense to be added to Essentials, especially not in the way that I've implemented it).

If possible I'd also be in my interest to switch back to the orginal Essentials (or maybe to a small fork where I just change the accessibility levels, as that would not break compatibility and enable me to implement my presets). I'll try to do that asap 👍 Is the issue still there regarding the string ids or am I missing something stupid there? :)

from theotherroles.

The issue is regarding UDP packet sizes, 0.2.0-dev.8 will address this within a few days.

from theotherroles.

Awesome, then I'll switch back with the new version ✔️ Thank you for your work.

from theotherroles.

0.2.0-dev.8 is now up, please let me know if you still experience a similar issue with the options not syncing, or any other issue you come across or feature you may need.

from theotherroles.

Thank you for the update, already switched locally 👍 I'm planning to still use a small variation of your Essentials, which only modifies 3 variablie accessibility levels (3x the ConfigEntry to pulbic). My use case is that I change the bind depending on a "preset option" (only for the settings that match my pluginId), so that I can store e.g. 4 different values for one option and switch between them by switching the preset setting. Do you still want me to create and link a fork there (as it doesn't affect compatibility at all and only three accessibilty levels changed) or are you fine with me just using that slight variation without pointing it out?

from theotherroles.

You can create derivatives of the option classes (in your mod) and add additional ConfigEntry fields for the presets, so you'd have one (the default) for the "active" preset and your additional ones to store presets, call SetValue to apply the selected preset.
And yes I'd like all modifications of Essentials to follow the license and post modifications of the code if any are made.

from theotherroles.