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

Moving the username to the side of the full name #161

Closed
wants to merge 1 commit into from

Conversation

@calebjonasson
Copy link
Contributor

commented Mar 24, 2014

Cleaning up the profile username on the user card.

Cleaning up the profile username on the user card.
@davedx

This comment has been minimized.

Copy link
Member

commented Mar 31, 2014

Hi there, sorry for the late reply and thank you for contributing to Solvers, it's appreciated!

It looks like your change broke one of the tests - probably because my test was too brittle ;) Could you have a look please? It's this one:

https://github.com/solvers/solvers/blob/master/test/profile.spec.js#L22

@calebjonasson

This comment has been minimized.

Copy link
Contributor Author

commented Mar 31, 2014

Will look into when home. Seems like a simple enough fix to update the test case.

@calebjonasson calebjonasson deleted the calebjonasson:patch-2 branch Aug 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.