-
Notifications
You must be signed in to change notification settings - Fork 100
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
add support for email confirmation #81
Conversation
I'm going to send appropriate PR to https://github.com/Jessecar96/SteamDesktopAuthenticator after(if) this PR will be merged. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't formally test it myself, but nothing sticks out immediately as strange except the difference in code style and indentation.
0e73fb1
to
ae630b6
Compare
SteamAuth/AuthenticatorLinker.cs
Outdated
if (!hasPhone) { | ||
if (confirmationEmailSent) { | ||
if (!_checkEmailConfirmation()) { | ||
return LinkResult.GeneralFailure; | ||
} | ||
} else if (!_addPhoneNumber()) { | ||
return LinkResult.GeneralFailure; | ||
} else { | ||
confirmationEmailSent = true; | ||
return LinkResult.MustConfirmEmail; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a huge issue, but you are using tabs to indent, which is inconsistent with the rest of the file which uses spaces.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, fixed it.
How can i access this ? |
@Burnwithme access what exactly? |
Is there an edited and ready to use version? |
@Burnwithme if you need fixed SteamDesktopAuthenticator - you can use my fork from here https://github.com/Ryzhehvost/SteamDesktopAuthenticator/releases/latest for now. If you need just library - you can clone my repo https://github.com/Ryzhehvost/SteamAuth for now. |
Recently steam changed procedure of adding authenticator - now before adding phone number it's required to confirm email once more. This PR adds support for this step.