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

Fix: remove unnecessary bypass permission #2060

Merged
merged 2 commits into from
Jan 15, 2023
Merged

Fix: remove unnecessary bypass permission #2060

merged 2 commits into from
Jan 15, 2023

Conversation

TheJonstone
Copy link
Contributor

Overview

This change will remove an unnecessary permission request

Description

The queries refer to fawe.bypass permission. This query is not necessary because the fawe.bypass.regions permission exists as a child in the fawe.bypass permission.

Submitter Checklist

  • Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
  • Ensure that the pull request title represents the desired changelog entry.
  • New public fields and methods are annotated with @since TODO.
  • I read and followed the contribution guidelines.

@TheJonstone TheJonstone requested a review from a team as a code owner January 4, 2023 21:06
@github-actions github-actions bot added the Bugfix This PR fixes a bug label Jan 4, 2023
@NotMyFault NotMyFault merged commit 0f55842 into IntellectualSites:main Jan 15, 2023
@TheJonstone TheJonstone deleted the fix/remove-unnecessary-bypass-permission branch January 16, 2023 15:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bugfix This PR fixes a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants