Skip to content

Admin acccess levels#10680

Merged
MaxIsJoe merged 4 commits into
unitystation:developfrom
Bod9001:admin-acccess-levels
Feb 7, 2025
Merged

Admin acccess levels#10680
MaxIsJoe merged 4 commits into
unitystation:developfrom
Bod9001:admin-acccess-levels

Conversation

@Bod9001

@Bod9001 Bod9001 commented Feb 1, 2025

Copy link
Copy Markdown
Member

Changelog:

CL: [New] new system for handling admins ranks and permissions

@github-actions github-actions Bot added the UI label Feb 1, 2025
@Bod9001 Bod9001 changed the title Admin acccess levels WIP Admin acccess levels Feb 6, 2025
@Bod9001 Bod9001 marked this pull request as ready for review February 6, 2025 23:33
@Bod9001

Bod9001 commented Feb 6, 2025

Copy link
Copy Markdown
Member Author

is not perfeect with showing on the client e.g the next round drop-down is still visible but I can't really think of a good way of handling it without making a bespoke Thing

and another good point with next round drop-down
should it be a separate Permission to see what a different admin has set next round drop-down , since that is currently lumped in with TAG.ADMIN_INFO that has loads of other stuff too

@Bod9001

Bod9001 commented Feb 7, 2025

Copy link
Copy Markdown
Member Author

NOTE 2
it doesn't have live updating,
since I don't know where to put it,
since it would need to access the logged in admins and refresh those and send out messages but also the path for permissions

fixed

@MaxIsJoe MaxIsJoe merged commit 807d68c into unitystation:develop Feb 7, 2025

public TAGType TAGType;

public List<TAGType> NeedsAny;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of two lists, you could do what we do in clearance code. It is basically the same logic

@Bod9001 Bod9001 deleted the admin-acccess-levels branch April 17, 2025 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants