Trigger note-webhook on comment edit
Problem
Currently, GitLab's note-webhooks are only triggered when comments are created. This limits the ability of external applications to react to and process comment updates.
Proposed Solution
Extend the existing note-webhook functionality to also trigger when comments within merge requests (or issues/snippets) are edited. This will provide a consistent mechanism for external systems to stay synchronized with the latest comment content.
Potential Use Cases
Workflow Automation:
Systems that rely on specific content or keywords within comments could trigger automated actions when a comment is updated.
This is the use case that I and RedHat are interested in.
External Integrations:
3rd party tools can remain tightly synchronized with GitLab discussions, even when comments undergo changes. This is the usecase @hussam789 and PRAgent are interested in.
Signed-off-by: Lucas Zampieri lzampier@redhat.com
Merge request reports
Activity
Hey @lzampier!
Thank you for your contribution to GitLab. Please refer to the contribution documentation for an overview of the process.
Did you know about our community forks? Working from there will make your contribution process easier. Please check it out!
When you're ready for a first review, post
@gitlab-bot ready
. If you know a relevant reviewer(s) (for example, someone that was involved in a related issue), you can also assign them directly with@gitlab-bot ready @user1 @user2
.At any time, if you need help, feel free to post
@gitlab-bot help
or initiate a mentor session on Discord. Read more on how to get help.You can comment
@gitlab-bot label <label1> <label2>
to add labels to your MR. Please see the list of allowed labels in thelabel
command documentation.This message was generated automatically. You're welcome to improve it.
added Community contribution workflowin dev labels
assigned to @lzampier
2 Warnings featureaddition and featureenhancement merge requests normally have a documentation change. Consider adding a documentation update or confirming the documentation plan with the Technical Writer counterpart.
For more information, see:
- The Handbook page on merge request types.
- The definition of done documentation.
This merge request does not refer to an existing milestone. Reviewer roulette
Category Reviewer Maintainer backend @jwanjohi
(UTC+0)
@harsimarsandhu
(UTC+5.5)
groupimport and integrate (backend) @SamWord
(UTC-4)
Maintainer review is optional for groupimport and integrate (backend) Please check reviewer's status!
Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.
If needed, you can retry the
danger-review
job that generated this comment.Generated by
DangerEdited by Lucas Zampierimentioned in issue gitlab-org/quality/triage-reports#13335 (closed)
added backend featureenhancement groupproject management typefeature labels
added devopsplan sectiondev labels
Setting label(s) Category:Team Planning based on groupproject management.
added Category:Team Planning label
added 870 commits
-
85d62687...b534caf9 - 869 commits from branch
gitlab-org:master
- 13a675e3 - Trigger note-webhook on comment edit
-
85d62687...b534caf9 - 869 commits from branch
added 3 commits
-
13a675e3...6d5fe525 - 2 commits from branch
gitlab-org:master
- f28df728 - Trigger note-webhook on comment edit
-
13a675e3...6d5fe525 - 2 commits from branch
@lzampier, it seems we're waiting on an action from you for approximately two weeks.
- Do you still have capacity to work on this? If not, you might want to close this MR and/or ask someone to take over.
- Do you need help in getting it ready? At any time, you can:
- If you're actually ready for a review, you can post
@gitlab-bot ready
.
This message was generated automatically. You're welcome to improve it.
added automation:author-reminded label
added 23459 commits
-
f28df728...4b836b80 - 23458 commits from branch
gitlab-org:master
- ad7bb68c - Trigger note-webhook on comment edit
-
f28df728...4b836b80 - 23458 commits from branch
added 4354 commits
-
ad7bb68c...55ee4f63 - 4353 commits from branch
gitlab-org:master
- 240ccf96 - Trigger note-webhook on comment edit
-
ad7bb68c...55ee4f63 - 4353 commits from branch
- Resolved by Carla Drago
Hi @lzampier, We are from PRAgent.
We are interested in getting a webhook when a Merge request comment (note) is edited or a checkbox is clicked, for implementing interactive MR actions (see similar ability in Github https://www.codium.ai/images/pr_agent/pr-actions.mp4).
- Is this kind of webhook already implemented? we tried, and we didn't receive an event when editing a comment.
- Will this MR enable it? Is there a timeline for merging it?
Thanks a lot Codium Team
Edited by hussam lawen
added 6684 commits
-
240ccf96...21aba482 - 6683 commits from branch
gitlab-org:master
- dae0f6be - Trigger note-webhook on comment edit
-
240ccf96...21aba482 - 6683 commits from branch