Fix authentication flow with Firebase integration across frontend and backend[Fix issue 1129]#1359
Open
G-venkateswarlu wants to merge 175 commits into
Open
Fix authentication flow with Firebase integration across frontend and backend[Fix issue 1129]#1359G-venkateswarlu wants to merge 175 commits into
G-venkateswarlu wants to merge 175 commits into
Conversation
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
…ontend/src/utils (aryandas2911#143) Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
…ryandas2911#63) Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
…ryandas2911#104) Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
…#116) Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
* fix(api): return 200 for empty task and routine lists * fix(ci): resolve ESLint compatibility issue * fix(api): return 200 for empty task list * fix(api): resolve merge conflicts and return 200 for empty task list --------- Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
…2911#1183) * feat: Added separate due date and due time on TaskFormModal * feat: made changes in the due-date-time feature --------- Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
…-builder (aryandas2911#562) Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
…1239) Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
…enation in endTime calculation (aryandas2911#1247) Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
* fix: parse startTime and duration as numbers to prevent string concatenation in endTime calculation * fix(backend): whitelist update fields in updateRoutine --------- Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
…as2911#1054) Co-authored-by: suryansh24-coder <soultionai@gmail.com> Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
* improve and consolidate troubleshooting section * Removed duplicate troubleshooting section --------- Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
) Closes aryandas2911#1221 Co-authored-by: Aryan Das <122392506+aryandas2911@users.noreply.github.com>
Removed console logs for API key and project ID.
Author
|
Hi @aryandas2911 👋 |
1 similar comment
Author
|
Hi @aryandas2911 👋 |
|
Hey @G-venkateswarlu! Saw your work on GSSoC 2026. We are building TermUI, a TypeScript terminal UI framework with React-style hooks and JSX, rendered entirely in the terminal. We have 74 unassigned GSSoC issues open across widgets, hooks, adapters, and CLI tooling. Your JavaScript background transfers directly. Karanjot, TermUI maintainer |
Owner
|
resolve conflicts |
Author
|
Hi @aryandas2911 👋 |
aryandas2911
requested changes
Jun 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
19 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📌 Description
Fixed issue #1129 by improving authentication flow across both frontend and backend. Updated Firebase integration, improved auth handling logic, and ensured proper configuration for secure login/signup functionality.
🔗 Related Issue
Closes #1129
🛠 Changes Made
-Updated backend authentication logic in backend/utils/firebaseAuth.js
-Improved Firebase Admin integration in backend
-Updated frontend authentication context in frontend/src/context/AuthContext.jsx
-Modified login flow in frontend/src/pages/Login.jsx
-Improved signup flow in frontend/src/pages/Signup.jsx
-Updated Firebase configuration in frontend/src/utils/firebase.js
-Updated .gitignore to improve project security and prevent sensitive files from being tracked
📸 Screenshots (if applicable)
Screen.Recording.mp4
☑️ Checklist
🚀 Notes for Reviewers
This PR improves authentication flow using Firebase across frontend and backend. Please review Firebase integration, auth state handling in React context, and backend verification logic.