-
Notifications
You must be signed in to change notification settings - Fork 164
Clarify single email for MFA #1984
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
Conversation
In a [recent internal discussion](https://elasticco.atlassian.net/browse/IAM-1778?focusedCommentId=251189), @alxchalkias shared his insights that we could update the doc to make it more clear that we only support one email factor. This PR is to address this context piece.
🔍 Preview links for changed docs: 🔔 The preview site may take up to 3 minutes to finish building. These links will become live once it completes. |
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.
LGTM!
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 for adding this!
I've left a very minor editorial suggestion, in case you'd like to implement it.
I'm totally ok with the change, but I'd like to highlight that ideally we should try to get rid of the entire FAQ at some point (I worked with the migration of this section of the docs, and I don't remember there was this big FAQ here 🤦 ) From a tech writing and communication perspective it would be much better to identify the key items of the questions and introduce them properly within the doc, so the most important items are correctly highlighted and the narrative makes more sense. But once said that, and considering we already have this FAQ, I'm ok with adding more questions until we decide to change it. cc: @shainaraskas PS - If @alxchalkias and @shainaraskas agree with it we could also create an issue to restructure this page and ensure we cover all items without the need of a FAQ, as after reading the questions I think it should be pretty straightforward. Of course this wouldn't be a task for @kunisen , but for us (core-docs team). |
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.
LGTM too!
Co-authored-by: Vlada Chirmicci <[email protected]>
🔍 Preview links for changed docs |
Thanks @eedugon! That said, I will go and merge this one, but feel free to tweak the structure to avoid FAQ style if needed. |
I agree we could restructure the page, but this isn't a very high prio thing to do, so I'd let core-docs team decide on the timing of this. |
Description
In a recent internal discussion, @alxchalkias shared his insights that we could update the doc to make it more clear that we only support one email factor.
This PR is to address this context piece.
Preview
cloud-account/multifactor-authentication.md