Skip to content

fix: OAuth connector callback redirect fix (backport of #2054)#2097

Merged
edwinjosechittilappilly merged 1 commit into
mainfrom
backport/2054-oauth-callback-redirect
Jul 15, 2026
Merged

fix: OAuth connector callback redirect fix (backport of #2054)#2097
edwinjosechittilappilly merged 1 commit into
mainfrom
backport/2054-oauth-callback-redirect

Conversation

@edwinjosechittilappilly

@edwinjosechittilappilly edwinjosechittilappilly commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Summary

Backports #2054 from release-saas-ga-0.6.2 to main.

Most of #2054's substance (the refreshAuth useCallback fix, CI workflow tweaks) was already merged to main independently via #2052. The only remaining diff after cherry-picking is a single explanatory comment in frontend/app/auth/callback/page.tsx that wasn't carried over.

Test plan

  • CI passes

Summary by CodeRabbit

  • Documentation
    • Added an inline clarification describing OAuth storage cleanup behavior.

* fix: OAuth connector callback redirect fix (#2052)

* fix connector callback

* React doctor improvements

* one more set of cleanups

* up ci version to match local npx runs

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

* style: apply biome auto-fixes [skip ci]

* address copilot review

---------

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* comment

---------

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions github-actions Bot added frontend 🟨 Issues related to the UI/UX bug 🔴 Something isn't working. labels Jul 14, 2026
@coderabbitai

coderabbitai Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 2bc700ae-0c0d-4171-8ed7-830a2ed89f49

📥 Commits

Reviewing files that changed from the base of the PR and between 7fb88fd and 52baf7b.

📒 Files selected for processing (1)
  • frontend/app/auth/callback/page.tsx

Walkthrough

Adds an inline comment above cleanupOAuthStorage() explaining that it removes OAuth-flow-related keys from localStorage. No functional logic changes.

Changes

OAuth cleanup documentation

Layer / File(s) Summary
Document OAuth storage cleanup
frontend/app/auth/callback/page.tsx
Adds a comment clarifying the purpose of cleanupOAuthStorage().

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: mfortman11

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the backported OAuth callback redirect fix and is specific enough to describe the PR's intent.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch backport/2054-oauth-callback-redirect

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added bug 🔴 Something isn't working. and removed bug 🔴 Something isn't working. labels Jul 14, 2026
@github-actions github-actions Bot added the lgtm label Jul 15, 2026
@edwinjosechittilappilly
edwinjosechittilappilly merged commit ac4b331 into main Jul 15, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🔴 Something isn't working. frontend 🟨 Issues related to the UI/UX lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants