-
Notifications
You must be signed in to change notification settings - Fork 16
feat: [AIDX-130] add notification channel settings content updates #189
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
base: feat/connected_accounts_doc_updates
Are you sure you want to change the base?
Conversation
| - **User's email verification status**: if the user's email is not verified, we assume sending emails to an unverified email address is not desired, and the request will be rejected. | ||
|
|
||
| You can read more about the notification channel selection in the [Configure Client-Initiated Backchannel Authentication](https://auth0.com/docs/get-started/applications/configure-client-initiated-backchannel-authentication) documentation. | ||
|
|
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.
This section is largely based on the internal doc here, and needs thorough review:
https://oktawiki.atlassian.net/wiki/spaces/ISGI/pages/3431662076/RFC+CIBA+with+Email+and+SMS#Notification-Channel-Selection
should a flow diagram also be added?
| - **User's enrolled authenticators**: if the user is not enrolled to use an MFA push authenticator, the current behavior is to fallback to Email instead of rejecting the request. | ||
| - **User's email verification status**: if the user's email is not verified, we assume sending emails to an unverified email address is not desired, and the request will be rejected. | ||
|
|
||
| You can read more about the notification channel selection in the [Configure Client-Initiated Backchannel Authentication](https://auth0.com/docs/get-started/applications/configure-client-initiated-backchannel-authentication) documentation. |
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.
are we planning to also enhance this page or any others related to CIBA within Auth0 docs for this @lrzhou25 ?
| Asynchronous Authorization | ||
| </a> | ||
| {" "} documentation. | ||
| </Step>, |
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.
Preference here is to keep updates minimal within the quickstarts, and link back to an overview page about this overfall flow / notification logic. My thinking is this logic could evolve and seems this would be an ok place to make reference to the configuration and allow them to dive further should it be necessary.
6340997 to
148636b
Compare
9381d2c to
770d1cb
Compare
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, would let Lucy also do another pass, but this is great 💪
Description
Adds some additional copy around CIBA Notification channels.
Some related PRs:
https://github.com/atko-cic/manhattan/pull/8138
https://github.com/atko-cic/auth0-server/pull/16424
References
JIRA: AIDX-130
Testing
Impacts the following pages:
Asynchronouis Authorization (overview page):
https://auth0-genai-async-auth-updates.mintlify.app/intro/asynchronous-authorization#user-consent-and-notification-channel-selection
Prerequisites (adds "Configure Notification Channel settings" section to all of the Async Authorization quickstarts):
https://auth0-genai-async-auth-updates.mintlify.app/get-started/asynchronous-authorization#prerequisites
Checklist