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.

Sign up for GitHub

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

Jump to bottom

docs: don't require a specific authenticator app #1251

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

docs: don't require a specific authenticator app #1251

wants to merge 1 commit into from

Conversation

gassims
Copy link

@gassims gassims commented Apr 11, 2023

Based on suggestions from @jaime Bosque and @amcgee. Lines 17 -18 need an update with the authenticator options.

@gassims
Not requiring a specific authenticator app
f9a52c8 
Based on suggestions from @jaime Bosque and @amcgee. Lines 17 -18 need an update with the authenticator options.
@amcgee amcgee changed the title DOCS: Not requiring a specific authenticator app docs: don't require a specific authenticator app Apr 11, 2023
@amcgee amcgee self-requested a review April 11, 2023 11:13
amcgee
amcgee approved these changes Apr 11, 2023
View reviewed changes
Copy link
Member

@amcgee amcgee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @gassims ! We could be a bit more broad and share more clear instructions for how to choose, find, and download an app but this is a great improvement over "requiring" a specific app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amcgee amcgee amcgee approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@gassims @amcgee