feedback not added when using (at least) 'Run the automated tools again'

Bug #1734910 reported by Jamie Strandboge
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Snap Store Server
Fix Released
Undecided
Facundo Batista

Bug Description

Snap came in for manual review due to classic confinement. I granted the use of classic via Overview/Review overrides. I then went to the revision that was up for manual review, filled in the feedback field with a comment mentioning I granted use of classic, then clicked 'Run the automated tools again'. The review started, but the comment was not added.

Due to https://bugs.launchpad.net/snapstore/+bug/1723475, I had to manually enter the URL to get to a form to enter feedback for the revision.

Tags: reviews
William Grant (wgrant)
tags: added: reviews
Revision history for this message
Adam Collard (adam-collard) wrote :

Jamie, is this still an issue?

Changed in snapstore:
status: New → Incomplete
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

Yes, it is.

Changed in snapstore:
status: Incomplete → New
Revision history for this message
Daniel Manrique (roadmr) wrote :

Would it be reasonable to modify the form's behavior so that when "run the automated review again" button is pressed, any input entered in the comment field gets submitted as a comment?

Sounds straightforward, I won't jinx the implementation more than that but wanted to clarify the expected behavior.

Changed in snapstore:
assignee: nobody → Facundo Batista (facundo)
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

That is what I expect for all of the buttons. There is a big textarea to put a comment-- if I put a comment in there and click Approve, Reject, Run the automated review again or whatever the 4th button is (I cleared the queue and can't find anything ;), then I expect that the comment is applied along with the action. It works that way with Reject, not sure why it doesn't for Run the automated review again....

Changed in snapstore:
status: New → In Progress
Changed in snapstore:
status: In Progress → Fix Committed
Changed in snapstore:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.