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: Always trust admin and moderators with post edits #25602

Merged
merged 1 commit into from Feb 8, 2024

Conversation

martin-brennan
Copy link
Contributor

Removes duplication from LimitedEdit to see who can edit
posts, and also removes the old trust level setting check
since it's no longer necessary.

Also make it so staff can always edit since can_edit_post?
already has a staff escape hatch.

Removes duplication from LimitedEdit to see who can edit
posts, and also removes the old trust level setting check
since it's no longer necessary.

Also make it so staff can always edit since can_edit_post?
already has a staff escape hatch.
martin-brennan added a commit to discourse/discourse-calendar that referenced this pull request Feb 8, 2024
This is needed for the change in the core PR
discourse/discourse#25602
martin-brennan added a commit to discourse/discourse-calendar that referenced this pull request Feb 8, 2024
This is needed for the change in the core PR
discourse/discourse#25602
@martin-brennan martin-brennan merged commit 7ce7614 into main Feb 8, 2024
19 checks passed
@martin-brennan martin-brennan deleted the issue/always-allow-staff-edit-post branch February 8, 2024 03:10
renato pushed a commit to renato/discourse that referenced this pull request Feb 29, 2024
Removes duplication from LimitedEdit to see who can edit
posts, and also removes the old trust level setting check
since it's no longer necessary.

Also make it so staff can always edit since can_edit_post?
already has a staff escape hatch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants