Skip to content

Fix: Login controller issue resolved & project cleanup#27

Open
kishan-semwal wants to merge 1 commit intoMERNRAD:mainfrom
kishan-semwal:fix/auth-improvements
Open

Fix: Login controller issue resolved & project cleanup#27
kishan-semwal wants to merge 1 commit intoMERNRAD:mainfrom
kishan-semwal:fix/auth-improvements

Conversation

@kishan-semwal
Copy link

✨ What I’ve Done in This PR:

✅ Fixed the login functionality by correctly validating user credentials (email & password)
✅ Implemented missing utility methods in User model:

  • setPassword
  • isValidPassword
    ✅ Cleaned up the codebase by replacing broken code with the working version
    ✅ Structured the project with proper client/server separation
    ✅ Added proper .gitignore files to exclude:
  • node_modules
  • environment variables (.env)
  • build artifacts
    ✅ Ensured admin login (testlibrarian@library.com) and member login work as expected
    ✅ Successfully tested all login scenarios using Postman

📌 This PR resolves the login issue and sets up the project for future enhancements with a clean and secure structure.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments