Skip to content

[codex] align remaining oauth docs and ai prompts#94

Merged
basit3407 merged 7 commits intomainfrom
docs/followup-oauth-prompt-fixes
Mar 31, 2026
Merged

[codex] align remaining oauth docs and ai prompts#94
basit3407 merged 7 commits intomainfrom
docs/followup-oauth-prompt-fixes

Conversation

@basit3407
Copy link
Copy Markdown
Collaborator

Summary

  • fix the remaining 'registered client' wording in the OAuth guide
  • align the Step 3 AI prompt with the frontend/native app plus backend exchange pattern
  • align the React Native AI prompt with the current discovery and logout examples

Testing

  • git diff --check
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the OAuth/OIDC documentation wording and the embedded “AI prompt” instructions to match the current recommended client patterns (frontend/native app + backend exchange) and current endpoint/discovery examples.

Changes:

  • Added token + revocation endpoints to the React Native AI prompt and explicitly calls out including them in the Expo discovery config.
  • Updated OAuth guide wording from “registered client” to “provisioned client type” and clarified code_verifier handling for different flow shapes.
  • Expanded acceptance checklists to reflect the updated guidance.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docs/tutorials/oidc/user-apis-quickstart.mdx Aligns the React Native AI prompt with the current discovery (auth/token/revoke) and logout/back-end exchange guidance.
docs/tutorials/oidc/getting-started-with-oauth2.mdx Updates admonition wording and clarifies Step 3 validation guidance for server-initiated vs app+backend exchange flows.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Mar 31, 2026

Deploying qf-api-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9a8f849
Status:⚡️  Build in progress...

View logs

@basit3407
Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/tutorials/oidc/getting-started-with-oauth2.mdx Outdated
Comment thread docs/tutorials/oidc/user-apis-quickstart.mdx Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/tutorials/oidc/getting-started-with-oauth2.mdx Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@basit3407 basit3407 merged commit a7d7187 into main Mar 31, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants