Closed
Bug 754441
Opened 13 years ago
Closed 13 years ago
Create admin featured tool
Categories
(Marketplace Graveyard :: Admin Tools, defect, P1)
Tracking
(Not tracked)
RESOLVED
FIXED
2012-06-07
People
(Reporter: clouserw, Assigned: ashort)
References
Details
Attachments
(3 files)
Our awesome featured manager based on collections has been deemed too simple to power the mighty apps marketplace. We'll need to build separate functionality for the marketplace, however, without the firefox/thunderbird/seamonkey/sunbird axis, I suspect it will be a lot easier to write and to use. Hope so anyway.
If an app doesn't have "Featured graphics" (which are added in the admin section of the app-edit pages) then they shouldn't be shown on the front end ever. You can stick a warning on the listing on the admin page if you think that'll be confusing later.
Sponsoring is a flag tied to the feature-status, not the app. Right now sponsoring means nothing - I've asked fligtar to clarify what it does.
I'm attaching a crude mockup I drew on the whiteboard which I hope answers more questions than it creates. Let me know if stuff doesn't make sense.
Spec: https://docs.google.com/document/d/1WX_Qis2qBDw3zrUSSmfmHSH2wZ4o-zAz2diD0wbovDA/edit
Reporter | ||
Updated•13 years ago
|
Assignee: nobody → ashort
Target Milestone: --- → 2012-05-24
Reporter | ||
Updated•13 years ago
|
Summary: Create admin featured tool :( → Create admin featured tool
Reporter | ||
Updated•13 years ago
|
Priority: -- → P1
Target Milestone: 2012-05-24 → 2012-06-07
Assignee | ||
Comment 1•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 2•13 years ago
|
||
'no Featured graphic' message should be in red. See screenshot
URL: https://marketplace-dev.allizom.org/en-US/admin/apps/featured
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 3•13 years ago
|
||
Reporter | ||
Comment 4•13 years ago
|
||
This is going to need work past changing the color to red. I was picturing this using the AMO style as a base (see attachment) but I didn't say that in here. I'll file bugs for the missing features here, but I'm going to close this one. Krupa: please verify just for basic functionality.
Reporter | ||
Updated•13 years ago
|
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•