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

more usersnap issues #469

Merged
merged 7 commits into from Mar 26, 2019
Merged

more usersnap issues #469

merged 7 commits into from Mar 26, 2019

Conversation

ascott
Copy link
Contributor

@ascott ascott commented Mar 22, 2019

  • update teaching text (was duplicated)
  • remove newsletter signup box in side nav, just use button
  • in create user flow, don't use user email as user name if one is not provided
  • on user pages, and article pages, truncate user names if it is an email address (so we don't display publicly)
  • replace copyright symbol with creative commons icon and link to license
  • add link “News” to medium in hamburger and footer

fixes:

@jesicarson @dethe

@ascott ascott merged commit d9a5741 into master Mar 26, 2019
@ascott ascott deleted the alanna-usersnap-issues-mar22 branch March 26, 2019 22:04
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.

None yet

1 participant