Feature #2882
openBulk comment submit
0%
Description
When user is reviewing the code, each inline comment is submitted and notification sent to users, this might be annoying. Phabricator has really nice sollution for that. See attached gif
Some specs:
- users should see their UNSUBMITTED comments in a summary view, and have ability to submit then (see second screenshot)
- Page should worn users before leaving that there are unsubmitted comments
- Unsubmitted comments should be stored inside DB, but they should not trigger notifications
- UN-submitted comments should be only visible to the creator (and super-admins)
- UN-submitted comments submit preview pane should be only visible to author, and super-admin. Both of them could submit the comments
- Email should be re-formatted with bulk comments, not to spawn multi-emails
UN-submitted comments should be marked with a tag, and show text:
- "this comment is private until you submit it" <submit it should be a link to the preview pane that allows submit
The preview pane should allow to delete comments, and re-create them (Edit them) ?
I think we Do Need EDIT functionality before this feature
Phabricator has this, and it's considered very nice option, also Kallithea has this ! Which we cannot be worse :)
Files
Updated by Marcin Kuzminski [CTO] about 9 years ago
- File bulk-submit-summary.png bulk-submit-summary.png added
Phabricator also creates main comments with links to inline comments submitted, which is kind of cool.
See screenshot bulk-submit-summary
Updated by Marcin Kuzminski [CTO] over 8 years ago
- Private changed from No to Yes
Updated by Marcin Kuzminski [CTO] over 8 years ago
- Private changed from Yes to No
It would be great to mimic the phabricator bulk comment submit functionality as it really helps to do reviews.
Main idea is that you submit comments as you move along in draft state, since code below can influence your comment above, once the review run is done you mark all submitted comments as done which triggers they final submition and notifies users.
- how do we do notifications in bulk ?
- how the UI should look
- how we need an legacy option to submit on comment