lp://staging/~michael.nelson/rnr-server/881051-ratings-with-6

Branch merges

Propose for merging

No branches dependent on this one.

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

Łukasz Czyżykowski (community): Approve on 2012-06-21

Diff: 367 lines (+231/-13)

8 files modified

requirements.txt (+3/-1)
setup.py (+1/-1)
src/reviewsapp/api/handlers.py (+1/-5)
src/reviewsapp/fields.py (+15/-1)
src/reviewsapp/migrations/0014_update_rating_field_with_range.py (+180/-0)
src/reviewsapp/models/reviews.py (+9/-5)
src/reviewsapp/tests/test_handlers.py (+7/-0)
src/reviewsapp/tests/test_models.py (+15/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

228. By Michael Nelson on 2012-06-21

Switched rating__lt=6 to rating__lte=5 for readability.

227. By Michael Nelson on 2012-06-20

Added migration simply updating ORM.

226. By Michael Nelson on 2012-06-20

GREEN: submitting a rating of 6 should be a bad request.

225. By Michael Nelson on 2012-06-20

RED: submitting a rating of 6 should be a bad request.

224. By Michael Nelson on 2012-06-20

GREEN: Don't error for ratings not in [0..5].

223. By Michael Nelson on 2012-06-20

RED: Don't error for ratings not in [0..5].

222. By Anthony Lenton on 2012-06-14

[r=michael.nelson] More pep8 fixes, and update pep8 test to fail when issue found.

221. By Anthony Lenton on 2012-06-14

[r=michael.nelson] PEP8 fixes.

220. By Anthony Lenton on 2012-06-12

[r=michael.nelson],[bug=778070] Made get_sca_publishings_for include apps published via extras.ubuntu.com and archive.canonical.com.

219. By Łukasz Czyżykowski on 2012-06-07

[r=lukasz-czyzykowski],[bug=744466] Added logout link.