Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 411 user profile name #455

Merged
merged 3 commits into from
Jul 4, 2019
Merged

Conversation

sssoleileraaa
Copy link
Contributor

Description

Resolves #411 without the ellipsis or hover-over overlay, which I believe @rmol is working on a generic solution for.

Here's an example of what this looks like (without styling) and only a first name:
fullname

Here's an example of what this looks like (again, without styling) and with a long first and last name:
long_name

Test Plan

Log in to the journalist interface and try setting different names to see initials and fullname change in user profile widget.

@sssoleileraaa sssoleileraaa force-pushed the issue-451-update-user-on-auth-success branch from f82668d to 04f47a8 Compare July 2, 2019 01:07
@sssoleileraaa sssoleileraaa changed the base branch from issue-451-update-user-on-auth-success to master July 2, 2019 01:12
@sssoleileraaa sssoleileraaa force-pushed the issue-411-user-profile-name branch from bf2e4c3 to f795852 Compare July 4, 2019 00:51
@sssoleileraaa sssoleileraaa marked this pull request as ready for review July 4, 2019 00:51
@sssoleileraaa sssoleileraaa force-pushed the issue-411-user-profile-name branch from f795852 to 4740654 Compare July 4, 2019 00:56
Copy link
Contributor

@redshiftzero redshiftzero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great!

@redshiftzero
Copy link
Contributor

(note that #285 is for the additional styling here)

@redshiftzero redshiftzero merged commit 6c8118d into master Jul 4, 2019
@redshiftzero redshiftzero deleted the issue-411-user-profile-name branch July 4, 2019 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show first name/last name and initials in AuthWidget if set
2 participants