Skip to content

docs: clarify redirect URI registration as recommended, not required#13933

Merged
rickyrombo merged 3 commits intomjp-web-example-oauth-uploadfrom
copilot/sub-pr-13925
Mar 16, 2026
Merged

docs: clarify redirect URI registration as recommended, not required#13933
rickyrombo merged 3 commits intomjp-web-example-oauth-uploadfrom
copilot/sub-pr-13925

Conversation

Copy link
Contributor

Copilot AI commented Mar 16, 2026

The upload example README incorrectly described registering a redirect URI as "required" — the OAuth flow works without it. Registration is a best practice that enables server-side validation of the callback URL.

Changes

  • packages/web/examples/upload/README.md: Changed "This is required" → "This is recommended" for the redirect URI registration step

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@changeset-bot
Copy link

changeset-bot bot commented Mar 16, 2026

⚠️ No Changeset found

Latest commit: 1583ab9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI and others added 2 commits March 16, 2026 22:36
…d" in upload example README

Co-authored-by: rickyrombo <3690498+rickyrombo@users.noreply.github.com>
…"can validate"

Co-authored-by: rickyrombo <3690498+rickyrombo@users.noreply.github.com>
Copilot AI changed the title [WIP] [WIP] Address feedback on web upload example for OAuth PKCE API update docs: clarify redirect URI registration as recommended, not required Mar 16, 2026
Copilot AI requested a review from rickyrombo March 16, 2026 22:37
@rickyrombo rickyrombo marked this pull request as ready for review March 16, 2026 22:38
@rickyrombo rickyrombo merged commit 93504dc into mjp-web-example-oauth-upload Mar 16, 2026
3 checks passed
@rickyrombo rickyrombo deleted the copilot/sub-pr-13925 branch March 16, 2026 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants