lp://staging/~michael.nelson/rnr-server/dont-depend-on-appname-2

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://staging/rnr-server at revision 163

Anthony Lenton (community): Approve on 2011-03-22

Diff: 441 lines (+192/-36)

10 files modified

src/reviewsapp/api/handlers.py (+6/-2)
src/reviewsapp/forms.py (+1/-2)
src/reviewsapp/management/commands/populate.py (+2/-2)
src/reviewsapp/migrations/0008_remove_softwareitem_app_name.py (+159/-0)
src/reviewsapp/models/reviews.py (+1/-6)
src/reviewsapp/tests/factory.py (+2/-2)
src/reviewsapp/tests/matchers.py (+5/-4)
src/reviewsapp/tests/test_handlers.py (+8/-8)
src/reviewsapp/tests/test_rnrclient.py (+6/-7)
src/reviewsapp/utilities.py (+2/-3)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #737649: Remove app name completely

Medium

Fix Released

Link a bug report

Related blueprints

175. By Michael Nelson on 2011-03-18

Ensure we always include a blank app_name attribute in case the client depends on that key being present.

174. By Michael Nelson on 2011-03-18

Removed softwareitem.app_name

173. By Michael Nelson on 2011-03-18

Splitting changes into second branch.

172. By Michael Nelson on 2011-03-18

Removeed softwareitem.app_name.

171. By Michael Nelson on 2011-03-18

Removed app_name option from makeSoftwareItem.

170. By Michael Nelson on 2011-03-18

Updated get_for_app tests to use the review.app_name attribute.

169. By Michael Nelson on 2011-03-18

Fixes to get tests passing again.

168. By Michael Nelson on 2011-03-18

Reverted last migration until tests passing with 0007.

167. By Michael Nelson on 2011-03-18

Added last migration to remove app name

166. By Michael Nelson on 2011-03-18

More updates to get manual migration test passing.