Skip to content

fix: allow re-submitting rejected OAuth clients#28133

Closed
deepshekhardas wants to merge 1 commit intocalcom:mainfrom
deepshekhardas:fix/oauth-client-re-submission-clean
Closed

fix: allow re-submitting rejected OAuth clients#28133
deepshekhardas wants to merge 1 commit intocalcom:mainfrom
deepshekhardas:fix/oauth-client-re-submission-clean

Conversation

@deepshekhardas
Copy link

What does this PR do?

Allows OAuth client owners to re-submit their application after it has been rejected. When an owner updates any field on a REJECTED OAuth client, the status automatically transitions back to PENDING for re-review.

Changes

  • Added isOwnerUpdatingFields parameter to computeNextStatus()
  • When currentStatus === 'REJECTED' && isOwnerUpdatingFields → status returns to PENDING
  • Added test case verifying REJECTED → PENDING transition on owner field update

Why

Previously, once an OAuth client was rejected, the owner had no way to fix the issues and re-submit for approval. This left rejected clients in a permanent dead state.

Fixes

Resolves the inability for OAuth client owners to iterate on rejected applications.

@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Feb 23, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@romitg2
Copy link
Member

romitg2 commented Feb 23, 2026

@deepshekhardas as requested on your earlier pr, please attach video demo

@romitg2 romitg2 marked this pull request as draft February 23, 2026 05:27
@sahitya-chandra
Copy link
Member

Closing due to inactivity, previous comment is not addressed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Created by Linear-GitHub Sync size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants