fix: add missing logger import to authStore.ts to prevent ReferenceError on all auth operations (closes #2085) - #2091
Conversation
…ror (closes riteshbonthalakoti#2085) authStore.ts makes 15+ calls to logger.log(), logger.warn(), and logger.error() but never imported the logger utility. Every auth operation (login, signup, magic link, OTP, profile update) throws ReferenceError: logger is not defined, breaking all authentication. Fix: add import { logger } from '../utils/logger';
|
@namann5 is attempting to deploy a commit to the ritesh Team on Vercel. A member of the Team first needs to authorize it. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
riteshbonthalakoti
added
gssoc
|
Superb implementation, @namann5! I've successfully resolved all conflicts in your PR and queued it for merging into
Keep up the outstanding work! Let's build together! 🔥 |
19a00c3
into
riteshbonthalakoti:gssoc
2 participants
Description
Fixes a critical ReferenceError that breaks all authentication in the frontend.
The Bug
\Frontend/src/store/authStore.ts\ makes 15+ calls to \logger.log(), \logger.warn(), and \logger.error()\ across login, signup, magic link, OTP verification, and profile update code paths — but never imports a logger.
This throws \ReferenceError: logger is not defined\ on every auth operation, making the app completely unusable for authentication.
The project has a logger utility at \src/utils/logger.js\ that should be imported.
Fix
Added \import { logger } from '../utils/logger';\ to the imports in \�uthStore.ts.
Closes #2085