Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Updated OAuth docs regarding selection of application type #369

Closed
wants to merge 2 commits into from
Closed

Updated OAuth docs regarding selection of application type #369

wants to merge 2 commits into from

Conversation

Rohit-RA-2020
Copy link

What does this PR do?

Updates the doc regarding selection of application type in OAuth providers

Test Plan

Doc updates

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

Yes

@Rohit-RA-2020
Copy link
Author

Hey @gewenyu99, you can review this

app/views/docs/authentication.phtml Outdated Show resolved Hide resolved
Co-authored-by: Vincent (Wen Yu) Ge <wenyu1999@hotmail.com>
@@ -307,6 +307,11 @@ try await account.createOAuth2Session(provider: "amazon")</code></pre>
If there is already an active anonymous session, the new session will be attached to it. If there are no active sessions, the server will attempt to look for an account with the same email address as the email received from the OAuth2 provider and attach the new session to the existing account. If no matching account is found - the server will create a new account.
</p>

<div class="notice">
<h3>Note</h3>
Copy link
Contributor

Choose a reason for hiding this comment

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

On second thought, can you suggest some better titles for this than "Note"? We don't use generic titles like this.

Copy link
Contributor

@gewenyu99 gewenyu99 left a comment

Choose a reason for hiding this comment

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

See previous comment

@Rohit-RA-2020 Rohit-RA-2020 closed this by deleting the head repository Aug 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants