Revert OAuth changes from PR #112 - restore working authentication by ishaileshpant · Pull Request #115 · prepguides/prepguides.dev

ishaileshpant · 2025-10-08T18:35:42Z

🔄 Revert OAuth Changes from PR #112

Problem

PR #112 introduced OAuth authentication issues that broke the main branch:

OAuth redirect_uri is being truncated in the URL
Users getting "redirect_uri is not associated with this application" error
Authentication flow is completely broken

Solution

This PR reverts the changes from PR #112 to restore the working OAuth authentication state.

Changes

✅ Reverted commit 9754d7d - "Admin-approved merge: Fix OAuth authentication: Add missing redirect_uri parameter"
✅ Restored working OAuth flow from before PR Fix OAuth authentication: Add missing redirect_uri parameter #112
✅ Follows proper branch naming with fix/ prefix

Impact

🎯 Restores working OAuth authentication
🚀 Fixes the broken main branch
🔄 Reverts to the last known working state

Testing

OAuth authentication should work properly after this revert
No more "redirect_uri not associated" errors
Users can authenticate with GitHub successfully

Priority: HIGH - This fixes the broken authentication on main branch

Note: After this revert, we can investigate the OAuth issue more carefully and implement a proper fix.

…edirect_uri parameter" This reverts commit 9754d7d, reversing changes made to 2242350.

vercel · 2025-10-08T18:35:48Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
prepguides-dev	Ready	Preview	Comment	Oct 8, 2025 6:37pm

ishaileshpant · 2025-10-08T18:36:09Z

/approve

- Keep the original working OAuth code without redirect_uri parameter - This restores the working authentication state from before PR #112 - Resolves conflicts between revert and main branch changes

ishaileshpant · 2025-10-08T18:37:49Z

/approve

github-actions · 2025-10-08T18:38:00Z

🚀 Admin-Approved and Auto-Merged!

This PR has been automatically merged to main after admin approval by @ishaileshpant.

✅ **Status**: Merged and deployed
🔗 **Commit**: `bc6c2abae4c67750fc11587b99e188bc1fdf044e`
📅 **Merged at**: 2025-10-08T18:38:00.479Z
👤 **Approved by**: Admin/Maintainer/Owner

Thank you for your contribution! 🎉

Revert "Admin-approved merge: Fix OAuth authentication: Add missing r…

975078f

…edirect_uri parameter" This reverts commit 9754d7d, reversing changes made to 2242350.

vercel Bot deployed to Preview October 8, 2025 18:35 View deployment

Resolve merge conflicts in revert PR

83a881e

- Keep the original working OAuth code without redirect_uri parameter - This restores the working authentication state from before PR #112 - Resolves conflicts between revert and main branch changes

vercel Bot deployed to Preview October 8, 2025 18:37 View deployment

github-actions Bot merged commit bc6c2ab into main Oct 8, 2025
11 checks passed

ishaileshpant deleted the fix/revert-oauth-changes-from-pr-112 branch October 13, 2025 18:19

Navigation Menu

Revert OAuth changes from PR #112 - restore working authentication - #115

Revert OAuth changes from PR #112 - restore working authentication#115
github-actions[bot] merged 2 commits into
mainfrom
fix/revert-oauth-changes-from-pr-112

ishaileshpant commented Oct 8, 2025

Uh oh!

vercel Bot commented Oct 8, 2025 •

edited

Loading

Uh oh!

ishaileshpant commented Oct 8, 2025

Uh oh!

ishaileshpant commented Oct 8, 2025

Uh oh!

Uh oh!

github-actions Bot commented Oct 8, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

ishaileshpant commented Oct 8, 2025