Skip to content

Add API support for redirect URI in dev apps#700

Merged
rickyrombo merged 7 commits intomainfrom
mjp-redirect-uris
Mar 9, 2026
Merged

Add API support for redirect URI in dev apps#700
rickyrombo merged 7 commits intomainfrom
mjp-redirect-uris

Conversation

@rickyrombo
Copy link
Contributor

Also ran npm i and prettier on plans

Copy link
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

Adds backend/API support for storing and returning OAuth redirect/callback URIs for developer apps, along with Swagger schema updates. Also includes formatting-only changes in the Plans frontend and an updated package-lock.json from running npm i.

Changes:

  • Extend developer-app API models and endpoints to accept, persist, and return redirect_uris.
  • Update /v1/users/:userId/developer-apps SQL to include redirect URIs (and metrics variant).
  • Update Swagger schemas to document redirect_uris on developer app objects and request bodies.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
static/plans/src/components/DeleteAppModal.tsx Prettier formatting only.
static/plans/src/App.tsx Prettier formatting only.
static/plans/package-lock.json Lockfile updated after npm i (includes metadata reshuffling).
api/v1_users_developer_apps.go Adds redirect_uris field to structs, queries, and create/update persistence logic.
api/swagger/swagger-v1.yaml Documents redirect_uris in relevant schemas/request bodies.
Files not reviewed (1)
  • static/plans/package-lock.json: Language not supported

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

@rickyrombo rickyrombo merged commit e229fed into main Mar 9, 2026
5 checks passed
@rickyrombo rickyrombo deleted the mjp-redirect-uris branch March 9, 2026 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants