Skip to content

permissions iteration 1#9618

Merged
corp-0 merged 8 commits into
unitystation:developfrom
corp-0:permissions-it-1
Feb 18, 2023
Merged

permissions iteration 1#9618
corp-0 merged 8 commits into
unitystation:developfrom
corp-0:permissions-it-1

Conversation

@corp-0

@corp-0 corp-0 commented Feb 16, 2023

Copy link
Copy Markdown
Member

Purpose

Adds the new permissions system. It will allow server owners and administrators to set arbirtrary ranks in a config file and grant permissions to said ranks.

In this iteration, this is still not integrated into the game, but I wanted to PR as soon as possible because I don't know when will I have time to complete it.

Notes

Since this is still not visible by players, I'm not including a changelog record.

This is merely a first iteration and isn't yet integrated into the game.
@github-actions

Copy link
Copy Markdown
Contributor

Your PR has no registered changes in its description!

@corp-0

corp-0 commented Feb 16, 2023

Copy link
Copy Markdown
Member Author

Don't merge yet, I have to make some changes in the build script

@corp-0

corp-0 commented Feb 16, 2023

Copy link
Copy Markdown
Member Author

done. Let's see if this works for tests.

@corp-0 corp-0 added the Status: Awaiting Review for PRs that are awaiting reviews/rereviews label Feb 18, 2023
Comment thread UnityProject/Assets/StreamingAssets/admin/permissions.toml Outdated
@corp-0
corp-0 requested a review from ThatDan123 February 18, 2023 17:33
@corp-0 corp-0 removed the Status: Awaiting Review for PRs that are awaiting reviews/rereviews label Feb 18, 2023
@corp-0
corp-0 merged commit 3f877ce into unitystation:develop Feb 18, 2023
@corp-0
corp-0 deleted the permissions-it-1 branch February 18, 2023 18:11
@github-actions

Copy link
Copy Markdown
Contributor

Your PR was merged with no registered changes or there was a problem with the API caller.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants