lp:~aaronp/software-center/reviews-tweaks

Branch merges

Propose for merging

No branches dependent on this one.

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #713013: Launchpad "display name" and "name" confusion in reviewing process	Undecided	Fix Released
Bug #733366: Unspecified is not a tag but a clue to specify a reason for moderation	Undecided	Invalid
Bug #754879: No way to tell how to review previously reviewed software that's not installed	Undecided	Fix Released

Link a bug report

Related blueprints

1468. By Aaron Peachey on 2011-04-11

softwarecenter/view/widgets/reviews.py:
 - small tweak to ensure 'you must install' label doesn't stay visible while review spinner is visible
 - fix regression - don't display 'Write Another Review' if user has reviewed current app

1467. By Aaron Peachey on 2011-04-11

softwarecenter/view/widgets/reviews.py: put label in same place as 'new review' link usually is

1466. By Aaron Peachey on 2011-04-09

softwarecenter/view/widgets/reviews.py:
    - advise user that app needs to be installed in order to review,
      even if reviews exist (LP: #754879)

1465. By Aaron Peachey on 2011-04-09

merge from trunk

1464. By Aaron Peachey on 2011-03-13

utils/submit_review.py: Remove 'unspecified' reason and disable submit button if no reason selected for report abuse (LP: #733366)

1463. By Aaron Peachey on 2011-03-13

merge from trunk

1462. By Aaron Peachey on 2011-03-02

utils/submit_review.py: update to match spec (as discussed with mpt) - checkmark, success message and 2 second wait after any successful submit (whether gwibber involved or not)

1461. By Aaron Peachey on 2011-02-25

updated changelog (lp:713013)

1460. By Aaron Peachey on 2011-02-25

view/appetailsview_gtk.py: check if displayname is None before trying to use it (will allow this change to work with older version of server that doesn't give us the display name)

1459. By Aaron Peachey on 2011-02-25

db/reviews.py: get review display name now rnr server supports it
view/appdetailsview_gtk.py: use display name instead of username (revert to username if review record has an empty string for display name)
(this feature only currently working on rnr-server at 184.82.116.62)