If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Migrate Theme submission to Marketplace

RESOLVED FIXED in 2012-09-27

Status

Marketplace
Developer Pages
P1
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: cvan, Assigned: ngoke)

Tracking

(Depends on: 1 bug)

2012-09-27
Points:
---
Dependency tree / graph

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Created attachment 640009 [details]
Current Persona submission flow

Personas can currently be uploaded at https://addons-dev.allizom.org/en-US/developers/persona/submit

We want this to instead live at https://marketplace-dev.allizom.org/en-US/developers/persona/submit

Please use the Marketplace Developer Hub style. Ask cvan if you have questions.
Assignee: nobody → ngoke
Target Milestone: --- → 2012-08-30
Target Milestone: 2012-08-30 → 2012-09-06
(Reporter)

Comment 1

5 years ago
Kevin has this pretty much done. I've seen it.
Target Milestone: 2012-09-06 → 2012-09-20

Comment 2

5 years ago
Is the marketplace the best place for this? Would it make more sense to be in AMO? That would be the quickest path to getting rid of getpersonas.
(In reply to Andy McKay [:andym] from comment #2)
> Is the marketplace the best place for this? Would it make more sense to be
> in AMO? That would be the quickest path to getting rid of getpersonas.

I wasn't involved in the decision (I imagine Wil and Fligtar were) but I understand the reasoning was that as both Addons and Themes/Personas are eventually going to be on Marketplace, it didn't make sense to put the work to migrate to AMO, and then a few months later have to rewrite it to migrate to Marketplace.
(Assignee)

Comment 4

5 years ago
https://github.com/mozilla/zamboni/commit/7756c230b8ba1a98d02a37c90b659ae9fdc6b77f
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
(Assignee)

Comment 5

5 years ago
at /developers/submit/theme
Looks nice but unfortunately I couldn't get it to work.

This error (from the console) might be relevant:
[11:53:36.362] Form contains a file input, but is missing method=POST and enctype=multipart/form-data on the form.  The file will not be sent. @ https://marketplace-dev.allizom.org/developers/submit/theme

Also, when the submission didn't complete due to above (and I was returned to the same page) the licensing radio inputs didn't retain their values.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 7

5 years ago
https://github.com/mozilla/zamboni/commit/49f067d207fe6ef74c487793fb4ff4a0524d5f8e

Attempted fix.
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED

Comment 8

5 years ago
When there is a form validation error, i.e. when the name you pick is already taken, the form makes you upload the images again. This is a major pain point as it's sometimes necessary to try a few names before you get one that is unique.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---

Comment 9

5 years ago
Agree that it's a real pain from a usability standpoint.  Is a unique name really necessary when there is already a unique ID number?  If not, how much trouble would it be to fix this to allow a non-unique name?  Thanks.
(In reply to BrianZ from comment #9)
> Is a unique name really necessary when there is already a unique ID number?  

Yes.  You should be able to find something from its name without having to have an exact url.

Comment 11

5 years ago
Why would a search not allow you to find a persona without a unique name?  Words or titles should not be reserved for a single add-on.

Comment 12

5 years ago
In reply to comment #3.

I just feel the single and only focus should be shipping marketplace. The "merge" has been bumped so far into the future, who knows when it will happen. If we put personas onto amo, it will be there way, way sooner.
(Reporter)

Comment 13

5 years ago
(In reply to Andy McKay [:andym] from comment #12)
> In reply to comment #3.
> 
> I just feel the single and only focus should be shipping marketplace.

It still is, yeah

> The "merge" has been bumped so far into the future, who knows when it will
> happen. If we put personas onto amo, it will be there way, way sooner.

I think the product call was to move themes entirely to Marketplace (without add-ons), and launch both. With Kevin gone, we won't have anyone actively working on themes so this is likely to get neglected pretty soon.

Updated

5 years ago
Target Milestone: 2012-09-20 → 2012-09-27
(Reporter)

Updated

5 years ago
Assignee: ngoke → nobody
(Reporter)

Comment 14

5 years ago
771850(In reply to Amy Tsay [:amyt] from comment #8)
> When there is a form validation error, i.e. when the name you pick is
> already taken, the form makes you upload the images again. This is a major
> pain point as it's sometimes necessary to try a few names before you get one
> that is unique.

Fixed it (for both AMO+Marketplace) so that we properly remember header+footer images and text+background colors even when there are form errors:

https://github.com/mozilla/zamboni/commit/38a663f
https://github.com/mozilla/zamboni/commit/9b748c8

For any additional issues with the flow, please file new bugs :)
Assignee: nobody → ngoke
Status: REOPENED → RESOLVED
Last Resolved: 5 years ago5 years ago
Resolution: --- → FIXED
Has anyone actually gotten the submission verifiably to work?  

My attempts result in listings that don't have working images
E.g. https://marketplace-dev.allizom.org/theme/bophutbeach1/ shows http://dev2.addons.phx1.mozilla.com/img/uploads/personas/388169/preview.jpg (outside of a VPN it doesn't resolve; inside it 404's).
(In reply to Andrew Williamson [:eviljeff] from comment #15)
> Has anyone actually gotten the submission verifiably to work?  
> 
> My attempts result in listings that don't have working images
> E.g. https://marketplace-dev.allizom.org/theme/bophutbeach1/ shows
> http://dev2.addons.phx1.mozilla.com/img/uploads/personas/388169/preview.jpg
> (outside of a VPN it doesn't resolve; inside it 404's).

filed as Bug 795966
You need to log in before you can comment on or make changes to this bug.