Skip to content

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

Merged
antoinejaussoin merged 3 commits intoantoinejaussoin:developfrom
hieuwu:develop
Nov 9, 2020
Merged

fix: login error when github account does not have display name#165
antoinejaussoin merged 3 commits intoantoinejaussoin:developfrom
hieuwu:develop

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