lp://staging/~fgallina/rnr-server/clickpackage-flagging

Branch merges

Propose for merging

No branches dependent on this one.

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

Matias Bordese (community): Approve on 2014-08-20

Diff: 1366 lines (+1194/-22)

9 files modified

src/clickreviews/api/handlers.py (+44/-9)
src/clickreviews/api/urls.py (+23/-1)
src/clickreviews/forms.py (+113/-0)
src/clickreviews/migrations/0002_auto__add_clickpackagereviewflag__add_clickpackagemoderation__add_clic.py (+182/-0)
src/clickreviews/models.py (+86/-0)
src/clickreviews/tests/factory.py (+52/-5)
src/clickreviews/tests/test_forms.py (+257/-2)
src/clickreviews/tests/test_handlers.py (+183/-1)
src/clickreviews/tests/test_models.py (+254/-4)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

279. By Fabián Ezequiel Gallina on 2014-08-20

Inject __name__ to clickreviews flag resources to avoid attribute errors

278. By Fabián Ezequiel Gallina on 2014-08-15

Cleanup useless transaction.atomic calls and use get_or_create

277. By Fabián Ezequiel Gallina on 2014-08-15

Add translations for click flag forms error messages

276. By Fabián Ezequiel Gallina on 2014-08-15

Fix clickreview 0002 migration DateTimeField defaults

275. By Fabián Ezequiel Gallina on 2014-08-15

Tests cleanup

274. By Fabián Ezequiel Gallina on 2014-08-14

Implementation of flagging for click reviews and packages

This is the initial implementation for click packages and click
package reviews flagging. The implementation differs from what's
found in reviewsapp, as all validation logic is provided via forms.

Current handlers allow for flagging packages or reviews, and a same
user cannot flag the same object twice.

273. By Fabián Ezequiel Gallina on 2014-08-13

Remove TestToolsTestCase from clickreviews tests

272. By Fabián Ezequiel Gallina on 2014-08-12

Disable Automatic flagging for ClickPackages

Several reasons to get this done:
  + This was about to fail in stg/prod if we enabled such feature
  + This feature is currently disabled in both prod and stg
  + Removes yet another dependency of reviewsapp in clickreviews

271. By Fabián Ezequiel Gallina on 2014-08-12

Decoupled ModerationFlag features from reviewsapp.ReviewModerationFlag

270. By Fabián Ezequiel Gallina on 2014-08-12

Decoupled Moderation features from reviewsapp.ReviewModeration

This adds a new core app that will eventually hold all common
functionality between reviewsapp and clickreviews so that
interpendence of these two will be removed.