Closed Bug 1130652 Opened 8 years ago Closed 7 years ago
Add user review guidelines
We need to add content guidelines for users who rate apps on Marketplace. On the submission dialog (http://i.imgur.com/9FBR04T.png), add the following text: Please read the Review Guidelines for more details about rating apps. Reviews that do not meet these guidelines may be removed by our moderation team without notice. Link the words "Review Guidelines" to a new static page (preferred URL: https://marketplace.firefox.com/review_guide) Preferably, the design is based on this template: https://marketplace.firefox.com/privacy-policy Content for this page: https://docs.google.com/a/mozilla.com/document/d/1IZk48hhJ7coq5s3L7U82BqplDEEfIvDlQ_5UfHjhaqc/edit# (Scroll to the bottom half of doc)
The content you linked to is all about AMO and add-ons. Aside from that, I think this content belongs on MDN under https://developer.mozilla.org/en-US/Marketplace . We've just managed to clear out all the long-form guides, docs, and policies out of the AMO codebase and onto MDN. We should keep the same for Marketplace.
The Marketplace-specific content is farther down the page. I'm ok with it on MDN...however, can we still have the content and link on the submission dialog?
Yes. Can you give us a new URL?
Whiteboard: [good first bug][contribute]
This is a good first bug and Andy has volunteered to help. Let us know if you are in a hurry though.
Hi Wil.. Can I work on this ?
Sure - Andy is the mentor. You can find him on IRC at andym (or ask in #marketplace). This bug is all in the front end code (fireplace). Instructions for installing fireplace can be found at https://github.com/mozilla/fireplace/ . The template this string should be added on is https://github.com/mozilla/fireplace/blob/master/src/templates/ratings/add.html . You don't need to worry about the "static page" stuff anymore since Amy put this on MDN. I'm going to needinfo Liz who can help with the UI. This has changed slightly since Amy's screenshot and I don't see a great place on this screen to add that many words. Hopefully Liz will have good input. :) Will attach screenshots in a minute...
Hi Wil, I have setup Docker on my system, and the instance is coming up. But how do I set up sample apps? I need to setup sample apps so that I can click on review button and test my changes. PS: I will wait for Liz's input. For now, I just want to play around. Thanks, Ram Vaishnav
(In reply to Ram Dayal Vaishnav [:Ram] from comment #10) > Hi Wil, > > I have setup Docker on my system, and the instance is coming up. But how do > I set up sample apps? I need to setup sample apps so that I can click on > review button and test my changes. `` fig run --rm zamboni python manage.py generate_apps N `` where N is the number of apps you want created in your database.
Ram, I spoke with Joni, and we updated the copy. See the attached "Submit Review" modal for where the 2 kinds of copy go. As you work on this, let me know if you have any questions!
maybe we can get this on KTLO Chapter 2?
Thanks Liz, I will submit the PR soon or ask you if I have any questions. Sorry for the delay.
Added to KTLO Chapter 2 (once it exists)
Thanks. I didn't realize Ram was still working on this. Assigning back.
Assignee: nobody → vaishnav.rd
Severity: normal → enhancement
Status: NEW → ASSIGNED
Priority: -- → P5
Target Milestone: 2015-02-24 → ---
Kindly r? https://github.com/mozilla/fireplace/pull/1163 Thanks, Ram
Looks like this was merged in. Thanks Ram for the patch. https://github.com/mozilla/fireplace/commit/eb3e02b4ec6ade277188864f0cd8cbb1a5fa9e83
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2015-04-28
I have verified it in FF40(Win7) in marketplace.allizom.org. The message is displayed and the link is working. In the mock-up from comment 12 there is a new text in the "Update content here" section. At this moment in marketplace the message is: "Tell us what you love about this app" - Is this intended Postfix screencast: http://screencast.com/t/Si30o9Pk8 Also the link to the review guidelines is not displayed on mobile devices (I have tested in Android 4.2.1 and Flame FFOS 2.0)- Intended or not? Screencast: http://screencast.com/t/YPFj8Mw7 Will can you please respond?
You're right, Madalin. The screenshots in the PR had the updated text, but the final one does not. I'm not sure what happened, but I'll reopen this.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [good first bug][contribute] → [good first bug][contribute][ktlo]
I will update the PR as required.
Submitted another PR #1255 for putting back the placeholder text (which was removed as per previous discussion on PR #1163).
Created another PR (#1354) as previous one was r+wc.
Merged #1358. Closing the request, feel free to reopen if required. Thanks :)
Status: REOPENED → RESOLVED
Closed: 8 years ago → 7 years ago
Resolution: --- → FIXED
Verified as fixed in FF42(Win7) in marketplace-dev. The text is updated and everything works as expected. Closing bug.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.