Closed Bug 478088 Opened 11 years ago Closed 11 years ago

[W-1.5.3] User Profiles

Categories

(addons.mozilla.org Graveyard :: Collections, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: fligtar, Assigned: wenzel)

References

()

Details

(Whiteboard: [not blocked by clearleft])

Attachments

(2 files)

Per the spec, users can make a list of collections they own and their subscriptions available to the public in their user profile.
Summary: [W-1.5.3] [Design] User Profiles → [W-1.5.3] User Profiles
Target Milestone: BW-M3 → BW-M4
Attached image mockup, v1
Mockup of the new fields. I think the user profile page is in need of a more space and information efficient layout, but for now, we can just tack these onto the bottom.
This is ready for implementation.
Assignee: nobody → fwenzel
Moving to revised M5 milestone (4/28).
Target Milestone: BW-M4 → BW-M5
Collections by... are only the ones for which the user is the owner (not admin, not publisher), I assume.

What, however, are favorite collections? The ones they are subscribed to (again, not admin, not publisher)?
(In reply to comment #4)
> Collections by... are only the ones for which the user is the owner (not admin,
> not publisher), I assume.
> 
Owner and publisher. If the user has any role on that collection, it should appear.

> What, however, are favorite collections? The ones they are subscribed to
> (again, not admin, not publisher)?
>
Favorite collections are "subscriptions", yeah. We'll be calling them favorites everywhere on the site.
Attached patch Patch, rev. 1Splinter Review
This patch should do it. Note that it's based on the code from bug 478106, as the DB changes from over there are accessed here.
Attachment #374479 - Flags: review?
Attachment #374479 - Flags: review? → review?(rdoherty)
Status: NEW → ASSIGNED
Ryan, can you review this soon?
Target Milestone: BW-M5 → 5.0.6
Whiteboard: [not blocked by clearleft]
Comment on attachment 374479 [details] [diff] [review]
Patch, rev. 1

Looks good and works.
Attachment #374479 - Flags: review?(rdoherty) → review+
Checked this in to r25724.
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Keywords: push-needed
Resolution: --- → FIXED
This requires SQL to be run on preview before it's testable, right?  Checkboxes aren't persisting their checked state for me.
Depends on: 493320
Verified FIXED; I offer my own profile as proof: https://preview.addons.mozilla.org/en-US/firefox/user/235535 :-)
Status: RESOLVED → VERIFIED
removing "push-needed" from 105 AMO 5.0.6 bugs; filter on "I hate stephend!"
Keywords: push-needed
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.