Skip to content

Conversation

@obenland
Copy link
Member

@obenland obenland commented Jul 9, 2025

Follow up to c958876.
See #1908.

Proposed changes:

  • Limit cap removal to front-end and comment edit screen.

Testing instructions:

  • Go to '..'

@obenland obenland requested a review from pfefferle July 9, 2025 12:48
@obenland obenland self-assigned this Jul 9, 2025
Copilot AI review requested due to automatic review settings July 9, 2025 12:48
@obenland obenland added the Skip Changelog Disables the "Changelog Updated" action for PRs where changelog entries are not necessary. label Jul 9, 2025

This comment was marked as outdated.

@pfefferle
Copy link
Member

not possible to trash or spam the comment

@obenland obenland requested a review from Copilot July 9, 2025 12:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the map_meta_cap method to conditionally block editing of federated comments, aiming to allow quick links on the comment list admin screen.

  • Restricts the do_not_allow capability addition to non-admins and the admin comment list screen.
  • Wraps the original block logic in a more specific admin/screen check.

@obenland
Copy link
Member Author

obenland commented Jul 9, 2025

Thank you! 🙏

@obenland obenland merged commit 85f62fe into trunk Jul 9, 2025
13 checks passed
@obenland obenland deleted the update/comment-links branch July 9, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog Disables the "Changelog Updated" action for PRs where changelog entries are not necessary.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants