978222 - Update user UI for events to match other apps

Status: RESOLVED FIXED

(Webmaker Graveyard :: Events, defect)

Description

[inactivate account for kate]

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.

Comment 1

[inactivate account for kate]

Attachment: https://github.com/mozilla/webmaker-events-2/pull/19/files

Comment 2

[Gavin Suntop [:gvn]]

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!

Comment 3

[inactivate account for kate]

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.

Comment 4

[inactivate account for kate]

Oops sorry wrong review flag:P

Comment 5

[Gavin Suntop [:gvn]]

Comment on attachment 8384767 [details]
https://github.com/mozilla/webmaker-events-2/pull/19/files

Just need to add target blanks, then R+!