Conversation
@@ -170,3 +170,41 @@ def escalate_prerelease_permissions(app, validation, version): | |||
app, version, nobody, 'App uses prerelease permissions', | |||
'developers/emails/prerelease_escalation.ltxt', | |||
amo.LOG.ESCALATION_PRERELEASE_APP) | |||
|
|||
|
|||
class SeparatedValuesField(fields.Field): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd prefer to move this to mkt/developers/fields.py
(which doesn't exist yet). Also, fix the import issue (fields
is not imported)
0a5d7f2
to
75db70c
Compare
okay, this is finished for now. I could add more of course, at the risk of introducing more breakage and having a more confusing diff. |
249cd18
to
883009f
Compare
if value: | ||
self.value_list.append(base_field.clean(value)) | ||
|
||
return u', '.join(self.value_list) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is generally useful. I'd vote we put this in mkt/site/fields.py
as it's not just a devhub thing even if that's the only place it might be used now.
I took a look and everything looks pretty good. Just one comment about a Field I think should be in a more general location. |
883009f
to
dd842cb
Compare
@robhudson done. |
r+ from me |
bug 1042820 - moor tidy of apps/amo
Not all of it, by a long way, but the bits I could clearly see were used or unused. Also I figured I better push the progress so far while its still (semi-) understandable.