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

Provider access token #394

Open
wants to merge 9 commits into
base: oauth-providers
Choose a base branch
from

Conversation

Dksie09
Copy link

@Dksie09 Dksie09 commented Jul 12, 2023

What does this PR do?

Updates GitHub OAuth doc by adding a provider access token section

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

```
## Accessing the Provider Access Token
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you include full code examples for each platform like other examples?

## Accessing the Provider Access Token

After the user has been authorized and redirected back to your app, you can find their GitHub provider access token in the [session object](https://appwrite.io/docs/models/session) by accessing the `providerAccessToken` property. This access token can be used to make requests to the GitHub API on their behalf.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think more context is needed here, why would someone need to use this access token? I would start by explaining what they are.

@gewenyu99
Copy link
Contributor

@Dksie09 Thanks for adding these responses, did you get a chance to sanity check the code examples by running them quickly?

@Dksie09 Dksie09 requested a review from gewenyu99 July 19, 2023 14:23
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.

Small comment, rest looks good.

app/views/docs/oauth-providers/github.md Outdated Show resolved Hide resolved
Dksie09 and others added 4 commits July 20, 2023 06:38
Co-authored-by: Vincent (Wen Yu) Ge <wenyu1999@hotmail.com>
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.

None yet

2 participants