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.

Sign up for GitHub

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

Jump to bottom

Event Makers permissions test #1324

Merged
merged 3 commits into from
Nov 28, 2023
Merged

Event Makers permissions test #1324

merged 3 commits into from
Nov 28, 2023

Conversation

Repede
Copy link
Contributor

@Repede Repede commented Nov 27, 2023 •
edited by izac112
Loading

Totally untested code because testosterone.. Actually because testing would still end up on the same db so I'd have to make a new account.

Izac edit:

  • Allow EVENT to change the weather
  • Only people with mod or Admin should be able to view and edit / add player notes
  • Only people with mod or Admin should be able to view and interact with tickets
  • Disallow non admins from viewing / interacting with asay
  • Disallow non admins / mods from viewing / interacting with msay

@izac112 izac112 self-requested a review November 27, 2023 00:46
@izac112 izac112 self-assigned this Nov 27, 2023
@izac112 izac112 added the Needs Testing Needs to be tested to make sure this is right label Nov 27, 2023
@izac112 izac112 added enhancement New feature or request and removed DO NOT MERGE Needs Testing Needs to be tested to make sure this is right labels Nov 28, 2023
izac112
izac112 approved these changes Nov 28, 2023
View reviewed changes
Copy link
Collaborator

@izac112 izac112 left a comment

Choose a reason for hiding this comment

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

Tested and its all working as intended

@izac112 izac112 merged commit da9e6ca into master Nov 28, 2023
6 checks passed
@izac112 izac112 deleted the eventpermissionfix branch December 30, 2023 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@izac112 izac112 izac112 approved these changes

Assignees

@izac112 izac112

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Repede @izac112