Skip to content
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

CIC/MPs don't have access to change records #7986

Open
cm13-issue-bot opened this issue Jan 3, 2025 · 4 comments
Open

CIC/MPs don't have access to change records #7986

cm13-issue-bot opened this issue Jan 3, 2025 · 4 comments
Labels
Bug Admin events aren't bugs

Comments

@cm13-issue-bot
Copy link

Testmerges

#7909: 'Fixes maintainer access to the profiler and clears profiler access whenever admins are loaded.' by Watermelon914 at commit f10699e7c0
#6951: 'New "in_hands" and "onmob" icon sprites for objects, small changes and fixes' by LC4492 at commit 1af9bcfe4e
#7862: 'Proc Redefinition Linting' by Drulikar at commit f11c329002
#7850: 'Acid Runner: Acid gen cap' by TheManWithNoHands at commit b8d243882f
#7774: 'req console tgui' by harryob at commit a8c69c9dc5
#7955: 'TGUI Crew Manifest' by nauticall at commit 83fabc7934

Round ID

25721

Description of the bug

You cannot change the rank of someone in the records (like a survivor) to something else, it just says not enough access

What's the difference with what should have happened?

For it to be able to be changed

How do we reproduce this bug?

1: Become Commanding Officer (which should have AA on everything)
2: access security records
3: Attempt to change someone's 'rank' define on the records

Attached logs

N/A

Additional details

  • Author: NexusLeader
  • Admin: Stalkerino
  • Note: None
@cm13-issue-bot cm13-issue-bot added the Bug Admin events aren't bugs label Jan 3, 2025
@BartDrown
Copy link
Contributor

Yeah, unfortunately security records console is currently in sad state and needs overhaul.

BartDrown added a commit to BartDrown/cmss13 that referenced this issue Jan 5, 2025
feat: add record delete button, secure deleting all records

Fixes CIC/MPs don't have access to change records cmss13-devs#7986
@ItsVyzo
Copy link
Contributor

ItsVyzo commented Jan 16, 2025

Not a bug. If I recall only CO can do it.

@ItsVyzo ItsVyzo closed this as not planned Won't fix, can't repro, duplicate, stale Jan 16, 2025
@ItsVyzo ItsVyzo added Not a Bug help!!!! i shot the enemy and it didnt die and removed Bug Admin events aren't bugs labels Jan 16, 2025
@BartDrown
Copy link
Contributor

Not a bug. If I recall only CO can do it.

But in current state, even CO cannot edit that

@ItsVyzo
Copy link
Contributor

ItsVyzo commented Jan 20, 2025

Not a bug. If I recall only CO can do it.

But in current state, even CO cannot edit that

then that's a bug :p

@ItsVyzo ItsVyzo reopened this Jan 20, 2025
@ItsVyzo ItsVyzo added Bug Admin events aren't bugs and removed Not a Bug help!!!! i shot the enemy and it didnt die labels Jan 20, 2025
BartDrown added a commit to BartDrown/cmss13 that referenced this issue Jan 20, 2025
feat: add record delete button, secure deleting all records

Fixes CIC/MPs don't have access to change records cmss13-devs#7986
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Admin events aren't bugs
Projects
None yet
Development

No branches or pull requests

3 participants