Merge lp://staging/~robert-ancell/rnr-server/remove-reviewer-limit-for-clicks into lp://staging/rnr-server

Proposed by Robert Ancell
Status: Rejected
Rejected by: Fabián Ezequiel Gallina
Proposed branch: lp://staging/~robert-ancell/rnr-server/remove-reviewer-limit-for-clicks
Merge into: lp://staging/rnr-server
Diff against target: 18 lines (+1/-7)
1 file modified
src/reviewsapp/api/handlers.py (+1/-7)
To merge this branch: bzr merge lp://staging/~robert-ancell/rnr-server/remove-reviewer-limit-for-clicks
Reviewer Review Type Date Requested Status
Ratings and Reviews Developers Pending
Review via email: mp+280107@code.staging.launchpad.net

Commit message

Allow app reviews to be edited at any time - they can be easily deleted / re-created to do effectively the same and this matches the click behaviour.

Description of the change

Allow app reviews to be edited at any time - they can be easily deleted / re-created to do effectively the same and this matches the click behaviour.

This is motivated by the review implementation in GNOME Software - the UI would be much simpler if reviews could be easily modified. I noticed that clicks removed the limit (bug 1444041) for what seems to be the same reason.

To post a comment you must log in.
Revision history for this message
Fabián Ezequiel Gallina (fgallina) wrote :

We won't be doing any additional work on the .deb store, sorry!

Unmerged revisions

314. By Robert Ancell

Allow app reviews to be edited at any time - they can be easily deleted / re-created to do effectively the same and this matches the click behaviour

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches