-
Notifications
You must be signed in to change notification settings - Fork 3.5k
updated 2fa related docs removing totp refs #1740
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
Changes from 1 commit
f3d46a7
090de5d
297a501
acec5bf
03e2be7
ffa1daa
d4720af
85e004d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. note (no need to change in this PR): in a future it would be ideal to just remove this extra step as it's now an assertion rather than an option. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. you are correct ,now this 2fa method doesn't make any sense and we can take user directly to configure screen. i'll add a note could be a future task to also make respective change in the UI Edit; Added a note here - https://github.com/github/npm/issues/14221#issuecomment-3387848940 |
Uh oh!
There was an error while loading. Please reload this page.