Jorge Villalobos [:jorgev] (he/him) Reporter
	Description • 9 years ago

Firefox for Android will follow a similar schedule for signatures than Desktop, so we need to enable signatures on AMO for those extensions as well. As I understand it, this is what we need to do:

For all extensions on AMO that support Firefox for Android and don't support Desktop:

1) Enable signing when they pass review.
2) Enable unlisted extensions (not sure if this needs any action).
3) Run AMO script that auto-signs existing approved versions. This requires some wording changes to inform developers about the updated Android timeline.

Feel free to break these tasks into dependencies.

	Mathieu Agopian [:magopian]
	Comment 1 • 9 years ago

That sounds like a plan. I'll submit a PR soon, can you please show me what you'd like for the wording?

Here's the current wording (we could imagine have another email, or just changing the current wording to be for both): https://github.com/mozilla/olympia/blob/master/lib/crypto/tasks.py#L22-45

	Jorge Villalobos [:jorgev] (he/him) Reporter
	Comment 2 • 9 years ago

Actually, the email template looks good to go. We can use it as is, and I'll update the wiki to include Firefox for Android.

	Mathieu Agopian [:magopian]
	Comment 3 • 9 years ago

PR: https://github.com/mozilla/olympia/pull/766

	Mathieu Agopian [:magopian]
	Comment 4 • 9 years ago

When this goes live on production, we need to run the "./manage.py process_addons --task sign_addons" bulk signing script again, to sign all the Firefox for Android add-ons.

Adding a ni? to :jason so we don't forget

	[github robot]
	Comment 5 • 9 years ago

Commits pushed to master at https://github.com/mozilla/olympia

https://github.com/mozilla/olympia/commit/007a6f33954c80d1def86177219293e098e9fd50
Sign Firefox for Android add-ons (bug 1185789)

https://github.com/mozilla/olympia/commit/14eefa2e5b41147f6b810922afcaa9993c47c2a9
Merge pull request #766 from magopian/1185789-sign-firefox-for-android-addons

Sign Firefox for Android add-ons (bug 1185789)

	Mathieu Agopian [:magopian]
	Comment 6 • 9 years ago

Unmarking this as resolved, as we want to first test it on -dev and/or stage. Assigning to Krupa.

	Andy McKay
	Comment 7 • 9 years ago

Marking as resolved fixed so we can get qa on the case.