Skip to content
This repository has been archived by the owner on Aug 25, 2023. It is now read-only.

fix(): Fix nil pointer panic in verification example #302

Merged
merged 1 commit into from
Aug 31, 2020
Merged

fix(): Fix nil pointer panic in verification example #302

merged 1 commit into from
Aug 31, 2020

Conversation

obalunenko
Copy link
Contributor

After completing challenge Account at root structure stayed to be nil. That cases panic.
Assign Account from Challenge to root Account

@obalunenko
Copy link
Contributor Author

Could you please review @ahmdrz ?

@obalunenko
Copy link
Contributor Author

@ahmdrz
Or maybe it will be better to refactor login with verification challenge case logic?
Nevertheless, now it works without panic.

@felipeinf
Copy link
Collaborator

@ahmdrz
Or maybe it will be better to refactor login with verification challenge case logic?
Nevertheless, now it works without panic.

I tried it but didn't work

@ahmdrz ahmdrz merged commit b53d84f into ahmdrz:master Aug 31, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants