Skip to content

Comment mentions #5944

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
Open
ssddanbrown wants to merge 2 commits into
base: development
Choose a base branch
from
Open

Comment mentions #5944

ssddanbrown wants to merge 2 commits into from
+402 −8

Conversation

@ssddanbrown
Copy link
Member

@ssddanbrown ssddanbrown commented Dec 8, 2025

Related to #560.

Todo

  • Finish off editor UI.
  • Address remaining todos in code changes
  • Add lookup for users
    • Limit to those with ability to create/update comments for now.
  • Back-end comment parsing
  • Notification on mention
  • Notification preferences - Add option for comment mentions.
  • Test cases for back-end functionality.

Considerations

  • How do we handle mention changes on comment updates.
  • Do we need rate limiting to prevent spam like page update notifications.
  • Are these useful if we don't opt-in users to them by default?

Doc Updates

  • Update advisory - Anyone with comment create/update will technically be able to access a list of system users.

@ssddanbrown ssddanbrown self-assigned this Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@ssddanbrown ssddanbrown

Labels

💻 Front-End 🏭 Back-End 🛠️ Enhancement

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ssddanbrown