Permission changes should be included in codebase-changes

I’m aware this might not be the correct category, in which case this post may be moved without issue. It seemed the most appropriate category to me.

As a fork maintainer I recently had a very weird experience when our admins were suddenly unable to do a lot of things on the live server, even if those commands worked fine in testing.

This was because a permission was silently added to the available permissionlist, defaulting to off for all ranks (as it should), which meant a lot of commands which were moved behind that new permission were silently (bug report on that will be filed on GitHub but I did not receive a lacking permission message) unavailable.

I believe it would be beneficial that any merges that add new permissions or change/remove old ones are specifically pointed out in the codebase changes channel, such that fork maintainers aren’t caught off guard by this in the future when syncing with Wizard’s Den.

5 Likes

Added github

Related GitHub issue for the command feedback bug: Respawn is no longer a recognized command · Issue #28850 · space-wizards/space-station-14 · GitHub

This topic was automatically closed 30 days after the last reply. New replies are no longer allowed.