-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
A couple bug fixes #288
base: master
Are you sure you want to change the base?
A couple bug fixes #288
Conversation
02af1b3
to
295ab93
Compare
Converting to draft and will let it marinate on my server, since some fixes introduced new bugs (fixed them, but there might be more) |
Do you have any updates regarding this? Otherwise I'd cherry-pick some of these in the coming days. |
So far haven't noticed anything else broken with the basic functionality of adding/removing friends, setting friend as read only or read/write. Though setting a friend as a Manager is kinda broken, didn't test it much, but if a Manager opens Redstone settings menu and then closes the menu - all redstone settings become disabled Probably best to cherry-pick yeah EDIT: Oh oops, commented from my other GitHub account...awkward xD |
This pull request fixes a couple bugs - separated each fix in their own commit