Skip to content
This repository has been archived by the owner on Mar 15, 2018. It is now read-only.

flag app for rereview if rating changes to adult (bug 939220) #1409

Merged
merged 1 commit into from Nov 20, 2013

Conversation

ngokevin
Copy link
Contributor

  • alphabetize test factories and add simple user_factory
  • if rating changes to adult, flag and log.

@@ -646,6 +646,11 @@ class DELETE_USER_LOOKUP(_LOG):
format = _(u'User {0.name} {0.id} deleted via lookup tool.')
keep = True

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

line break pls

@cvan
Copy link
Contributor

cvan commented Nov 19, 2013

this looks good to me then. I'll let @robhudson give the official

r+

if rating.adult and not old_rating[0].get_rating().adult:
RereviewQueue.flag(
app, amo.LOG.CONTENT_RATING_TO_ADULT,
message=_('Content rating changed to Adult during daily '
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not necessarily a daily check they do. Maybe IARC spot check? Or just "Content rating changed to adult."?

@robhudson
Copy link
Member

r+wc thanks

@ngokevin
Copy link
Contributor Author

thanks

ngokevin added a commit that referenced this pull request Nov 20, 2013
flag app for rereview if rating changes to adult (bug 939220)
@ngokevin ngokevin merged commit b9a99d3 into mozilla:master Nov 20, 2013
@ngokevin ngokevin deleted the iarcadult branch November 20, 2013 20:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants