Skip to content

Allow developers to upload sources of an Addon version (bug 540028)#139

Merged
yohanboniface merged 1 commit into
mozilla:masterfrom
yohanboniface:bug-540028-add-version-source
Aug 6, 2014
Merged

Allow developers to upload sources of an Addon version (bug 540028)#139
yohanboniface merged 1 commit into
mozilla:masterfrom
yohanboniface:bug-540028-add-version-source

Conversation

@yohanboniface
Copy link
Copy Markdown
Contributor

Do not review yet (but comments always welcome ;) ).

Ref: https://bugzilla.mozilla.org/show_bug.cgi?id=540028

TODO

  • restrict source file access
  • change file link in form widget (or do we want another widget than FileField?)
  • add field to "Upload New Version" form
  • add field to "Submit a new addon" form
  • add field to "Upload a new file" form
  • add download link in the "editor page"
  • automatically flag addon for 'admin review' when a source is added
  • style forms

From Jorgev:

  • In the review page, the Download Source option should only be available to Admin Reviewers (group 50005). I would also use "Additional sources" so it's clearly distinguishable from the sources in the XPI.
  • In the upload forms, the source code form should appear after the upload and validation for the XPI succeeds. Otherwise it might confuse developers.
  • I would change the label "If your add-on contains binary or obfuscated code other than known libraries, upload its sources for review. Read more about the source code review policy." The 'Read more' text should link to https://addons.mozilla.org/developers/docs/policies/reviews#section-binary and open in a new tab.

Comment thread apps/devhub/forms.py Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I thought the cleaned_data would always have a 'source' key(?)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You're right. And BTW, .get(xxx, None) doesn't make sense as it's the default behaviour :p

@yohanboniface yohanboniface changed the title [WIP] Allow developers to upload sources of an Addon version (bug 540028) Allow developers to upload sources of an Addon version (bug 540028) Jul 5, 2014
@yohanboniface
Copy link
Copy Markdown
Contributor Author

We need to land #110 before merging here

@davidbgk
Copy link
Copy Markdown
Contributor

davidbgk commented Jul 5, 2014

r+ when ready for landing

yohanboniface added a commit that referenced this pull request Aug 6, 2014
Allow developers to upload sources of an Addon version (bug 540028)
@yohanboniface yohanboniface merged commit 0d73de1 into mozilla:master Aug 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants