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.

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

📚 Documentation: Missing example for accountCreateOauth2Session (GraphQL) #5082

Closed
2 tasks done
NobleUpFront opened this issue Feb 5, 2023 · 6 comments
Closed
2 tasks done
Labels
product / docs Fixes and upgrades for the Appwrite Docs.

Comments

@NobleUpFront
Copy link

💭 Description

Graphql documentation is missing an example for "accountCreateOAuth2Session"

https://appwrite.io/docs/client/account?sdk=graphql-default#accountCreateOAuth2Session

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

@NobleUpFront
Copy link
Author

Update:

Its not supported, leaving the issue open, such that the documentation can be updated.

@stnguyen90 stnguyen90 added the product / docs Fixes and upgrades for the Appwrite Docs. label Feb 6, 2023
@stnguyen90
Copy link
Contributor

Its not supported, leaving the issue open, such that the documentation can be updated.

@gewenyu99, what do you think about adding some sort of documentation to mention it's not supported?

@gewenyu99
Copy link

Its not supported, leaving the issue open, such that the documentation can be updated.

@gewenyu99, what do you think about adding some sort of documentation to mention it's not supported?

Sure, I can add in the tab and mention that it's not supported.

@abnegate
Copy link
Contributor

abnegate commented Feb 8, 2023

@stnguyen90 @gewenyu99 We do mention that oauth isn't available at the top of the page here: https://appwrite.io/docs/graphql, but the confusion is that there's no example here in the references here: https://appwrite.io/docs/client/account?sdk=graphql-default#accountCreateOAuth2Session. The linked PR is only updating the auth guide (which we should still do as well)

@gewenyu99
Copy link

@stnguyen90 @gewenyu99 We do mention that oauth isn't available at the top of the page here: https://appwrite.io/docs/graphql, but the confusion is that there's no example here in the references here: https://appwrite.io/docs/client/account?sdk=graphql-default#accountCreateOAuth2Session. The linked PR is only updating the auth guide (which we should still do as well)

😬 I see, maybe we should default empty templates to something that says this endpoint is not available with the selected version and SDK combination. Let me take a looksies

@gewenyu99
Copy link

I'm going to close this. We've added a notice saying that OAuth2 isn't supported for GraphQL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
product / docs Fixes and upgrades for the Appwrite Docs.
Projects
None yet
Development

No branches or pull requests

4 participants