Page MenuHomePhabricator

Unable to simply mark a Diffusion comment as Done
Closed, ResolvedPublic

Description

In Diffusion, if someone raised a concern on one of my commits and made a comment on a section of code:

  • I can click the "Done" checkbox by the comment to mark it as done.
  • This adds a note that my change is "Unsubmitted." This seems reasonable, and it makes me think I should click Submit at the bottom of the page.
  • If I click Submit at the bottom of the page, the page is submitted, and the server acts like it's doing something, but my marking the comment as Done still shows as unsubmitted.

Apparently I have to actually enter text in the Comment section at the bottom of the page for my change to be processed. If I don't have anything to say, it would be nice to simply click a few Done checkboxes, click Submit, and be done.

Event Timeline

I'm having trouble reproducing this -- I attempted to follow your steps, but didn't experience the issue. Here's what I did:

First, I loaded a revision that I had authored which another user had added an inline comment to:

Screen Shot 2016-02-24 at 8.18.41 AM.png (1×1 px, 137 KB)

Next, I clicked "Done":

Screen Shot 2016-02-24 at 8.18.44 AM.png (1×1 px, 140 KB)

Then, I clicked "Submit":

Screen Shot 2016-02-24 at 8.18.47 AM.png (1×1 px, 138 KB)

Finally, I confirmed that I didn't want to post any comments. The page reloaded, showing the inline now marked "Done":

Screen Shot 2016-02-24 at 8.18.53 AM.png (1×1 px, 145 KB)

Additionally, a transaction populated in the transaction log recording my change:

Screen Shot 2016-02-24 at 8.21.30 AM.png (122×2 px, 23 KB)

What can I do differently to reproduce this issue?

Your steps match mine. However, I'm not getting an "Action With No Effect" prompt; instead, the server acts like it's processing, and redisplays the page, but doesn't actually make any changes.

We're using Audit -> Diffusion (i.e., post-commit reviews). I just tested with a review through Differential, and I get the Action With No Effect prompt and can successfully mark as Done, as in your steps.

Ah, I missed "In Diffusion". Let me see if I can reproduce there.

avivey renamed this task from Unable to simply mark a code review comment as Done to Unable to simply mark a Diffusion comment as Done.Feb 28 2016, 7:47 PM
avivey added a project: Audit.

I believe D17183 fixes this. If you're still seeing issues after upgrading past that change, file a new issue with modern reproduction instructions.