feat(mr review): editing comments on MR diffs
This MR allows users to edit comments on MR diff.
Editing
API failure when submitting the changed comment
Somebody changed the comment since we fetched it
The user experience when this happens is far from ideal. We pretty much tell the user to:
- edit the comment again (so they can see their text that we failed to update)
- copy that comment to the clipboard
- make sure the side panel is open
- call a command to refresh the sidebar to see the latest discussion
The error handling prevents any data loss, but the process is a bad user experience.
This never happened to me when commenting on GitLab and so I'm not worried too much about the UX.
Related to #317 (closed)
Edited by Tomas Vik (OOO back on 2024-09-23)