-
Notifications
You must be signed in to change notification settings - Fork 1.1k
feat: implement user tagging in incident comments #4699
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
feat: implement user tagging in incident comments #4699
Conversation
|
@Kunal-Darekar is attempting to deploy a commit to the KeepHQ Team on Vercel. A member of the Team first needs to authorize it. |
|
Hey! |
ae8d96a to
ecb318d
Compare
ecb318d to
285d531
Compare
keep-ui/app/(keep)/incidents/[id]/activity/ui/MentionsInput.tsx
Outdated
Show resolved
Hide resolved
keep-ui/app/(keep)/incidents/[id]/activity/ui/QuillMentionsInput.tsx
Outdated
Show resolved
Hide resolved
|
Hi @Kunal-Darekar |
…ions. Rename QuillMentionsInput to CommentInput and remove redundant MentionsInput component.
|
@skynetigor please have a look and let me know if anything else is needed. |
…ationsProvider and ManualRunWorkflowModal imports
|
@Kunal-Darekar Stop stealing screenshots please |
@hemangsk , I sincerely apologize for including your PR screenshot in my submission. This was completely unintentional and an oversight on my part. I've removed the image and will be more careful with my PR documentation going forward. Thank you for bringing this to my attention. |
|
@Kunal-Darekar i'm closing this in favor of this first PR that was opened: #4649. If it will be rejected, we'll re-open yours. Sorry for the inconvenience. |



Description
This PR adds user tagging functionality to incident comments, allowing users to mention other team members in comments and send them notifications.
Features
Technical Implementation
Testing
/fixes [🔨 Enhancement]: Allow tagging in Incident comment #2187
/claim [🔨 Enhancement]: Allow tagging in Incident comment #2187