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

fix: login error when github account does not have display name #165

Merged
merged 3 commits into from
Nov 9, 2020

Conversation

hieuwu
Copy link
Contributor

@hieuwu hieuwu commented Nov 9, 2020

When using Github account without name to log in the application. Server thows an error like this. I fixed this issue by check if that account has name or not then set default name.

error-github-without-name

Account can log in:
true-profile

Account can NOT login:
false-profile

@hieuwu hieuwu closed this Nov 9, 2020
@hieuwu
Copy link
Contributor Author

hieuwu commented Nov 9, 2020

Close for wrong logic. Will fix in next pull request

@hieuwu
Copy link
Contributor Author

hieuwu commented Nov 9, 2020

Reopen after fixing logic

@hieuwu hieuwu reopened this Nov 9, 2020
@antoinejaussoin
Copy link
Owner

Thank you so much for this!

@antoinejaussoin antoinejaussoin merged commit 85fc4fa into antoinejaussoin:develop Nov 9, 2020
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.

2 participants