Closed
Bug 978222
Opened 11 years ago
Closed 11 years ago
Update user UI for events to match other apps
Categories
(Webmaker Graveyard :: Events, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kate, Assigned: kate)
Details
(Whiteboard: [events2][mvp])
Attachments
(1 file)
Right now only the email is shown. We should show the username, avatar, and a drop down to my makes and settings for the user's account.
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → kate
Assignee | ||
Updated•11 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #8384767 -
Flags: review?(gavin)
Comment 2•11 years ago
|
||
Comment on attachment 8384767 [details]
https://github.com/mozilla/webmaker-events-2/pull/19/files
For some reason I have to click logout twice in the dropdown before I actually get logged out.
Also, My Makes and Settings didn't get populated with URLs for either of my accounts that I tried.
Also also, looks like you need a rebase.
Other than that...cool!
Attachment #8384767 -
Flags: review?(gavin) → review-
Assignee | ||
Comment 3•11 years ago
|
||
Comment on attachment 8384767 [details]
https://github.com/mozilla/webmaker-events-2/pull/19/files
The logout issue was due to something that changed in the auth client, fixed in an earlier commit:
https://github.com/mozilla/webmaker-events-2/commit/46b1f3de80c000ad6196ae056da45aa1535eeb75
You should be getting webmaker.org/me and login.webmaker/account by default, are those not working for you? https://github.com/mozilla/webmaker-events-2/pull/19/files#diff-45c4a8fcc6031017d58405007de49792R17
They have to be configured in the server so that we can have the correct links for different environments (staging, production) -- I added some docs.
Attachment #8384767 -
Flags: review- → review+
Assignee | ||
Updated•11 years ago
|
Attachment #8384767 -
Flags: review+ → review?(gavin)
Assignee | ||
Comment 4•11 years ago
|
||
Oops sorry wrong review flag:P
Comment 5•11 years ago
|
||
Comment on attachment 8384767 [details]
https://github.com/mozilla/webmaker-events-2/pull/19/files
Just need to add target blanks, then R+!
Attachment #8384767 -
Flags: review?(gavin) → review+
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•