948656 - With Content ratings, it is not clear to developers how to get their apps to pending review status

Status: VERIFIED FIXED

(Marketplace Graveyard :: Developer Pages, defect, P1)

Description

[krupa raj[:krupa]]

Filing a new separate bug just keep issues separate. See related bugs https://bugzilla.mozilla.org/show_bug.cgi?id=939317 and https://bugzilla.mozilla.org/show_bug.cgi?id=948098

steps to reproduce:
1. Submit a paid app
2. In the Next Steps page, click on the link to get Content ratings
3. Complete the necessary ESRB questionnaire to get a rating

expected behavior:
After completing the content rating step, developer is encourage to set up Payments

observed behavior:
User is shown a success message and may end up thinking that their app submission is now complete resulting in a lot of incomplete app.s

Comment 1

[Christopher Van Wiemeersch [:cvan]]

This seems like a big deal

Comment 2

[Kevin Ngo [:ngoke]]

Attachment: devhub-iarc-mock

Here's a mock pertaining to this. It's a bit of a change.

What do you think about it? Is it clear enough, or should we look for an alternative flow?

Comment 3

[Kevin Ngo [:ngoke]]

Attachment: Screen Shot 2013-12-10 at 5.14.02 PM.png

Here's a quick fix of having a message up top "You must set up Content Ratings before we can review your app.".

Ideally, a developer would also get a success message "Your app is now pending status." or something once all is done. But given that the flow will undergo changes, I'll leave that off the table so it won't have to be rewritten.

Comment 4

[Christopher Van Wiemeersch [:cvan]]

"before we can review your app" does not necessarily mean that "your app cannot and will not be public until you do this." We should change the wording to make that explicit.

Comment 5

[Kevin Ngo [:ngoke]]

Hmm...

"You must set up Content Ratings else your app cannot be reviewed nor published."

or

"You must set up Content Ratings before your app cannot be reviewed or published."

?

Comment 6

[Kevin Ngo [:ngoke]]

"You must set up Content Ratings before your app can be reviewed or published."

Whoops.

Comment 7

[Wil Clouser [:clouserw]]

We'll solve this this afternoon. (this)

Comment 8

[Kevin Ngo [:ngoke]]

https://github.com/mozilla/zamboni/pull/1523

A small step to make it a bit clearer for now.

Comment 9

[Kevin Ngo [:ngoke]]

Bug relating to success message up top.

> Ideally, a developer would also get a success message "Your app is now pending status." or something once all is done. But given that the flow will undergo changes, I'll leave that off the table so it won't have to be rewritten.

Comment 10

[Tony Santos [:tsmuse]]

For the case when the rating is all that is required the success message should be, "Congratulations, your app submission is now complete and your app will be reviewed shortly!"

For the case when the rating is complete but payment account details are still outstanding the message should be, "Your app now has a rating. You still need to provide payment details before we can review your submission." and "payment details" should be a link to the payment details page. 

Hopefully that gets us to a good place on this until we can do a larger redesign of the flow.

Comment 11

[Kevin Ngo [:ngoke]]

https://github.com/mozilla/zamboni/pull/1533

I kept the copy "You still need to set up Payments...<etc>" for code simplicity. Let me know if that's okay. It might be worthwhile looking at the screenshots in that pull request link above.

Comment 12

[Tony Santos [:tsmuse]]

Looks good to me :)

Comment 13

[Iulian Timis]

Verified as fixed in https://marketplace-dev.allizom.org/developers/ on FF29 (Win 7).
Attaching postfix screenshot.
Closing bug.

Comment 14

[Iulian Timis]

Attachment: postfix_payment_request.png