Skip to content

[5] Auth: Implement merchant registration and authentication#110

Open
amehsamuel200225-sketch wants to merge 1 commit intoMissBlue00:mainfrom
amehsamuel200225-sketch:fix/5-auth-implement-merchant-registration-and-authentication
Open

[5] Auth: Implement merchant registration and authentication#110
amehsamuel200225-sketch wants to merge 1 commit intoMissBlue00:mainfrom
amehsamuel200225-sketch:fix/5-auth-implement-merchant-registration-and-authentication

Conversation

@amehsamuel200225-sketch
Copy link
Copy Markdown

@amehsamuel200225-sketch amehsamuel200225-sketch commented Mar 24, 2026

Closes #5

Summary

  • Merchant registration (\POST /auth/register) with email, password, and business name
  • Login (\POST /auth/login) returning JWT \�ccess_token, \expires_in, and public merchant profile
  • Current merchant (\GET /auth/me) for authenticated requests
  • \MerchantsService\ in-memory store (ready to swap for persistence later)
  • \JwtStrategy\ validates \merchant_id\ against registered merchants
  • Global \ValidationPipe\ for DTO validation

Env: \JWT_SECRET\ (required in production), optional \JWT_EXPIRES_IN_SEC\ (default 7 days).

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 24, 2026

Someone is attempting to deploy a commit to the missblue00's projects Team on Vercel.

A member of the Team first needs to authorize it.

- Add MerchantsService with in-memory store (ready to swap for DB)
- POST /auth/register and POST /auth/login with bcrypt password hashing
- GET /auth/me for authenticated merchant profile
- Wire JwtStrategy to verify merchant_id exists
- Global ValidationPipe for DTO validation
- Unit tests for AuthService and AuthController

Closes MissBlue00#5

Made-with: Cursor
@amehsamuel200225-sketch amehsamuel200225-sketch force-pushed the fix/5-auth-implement-merchant-registration-and-authentication branch from 9510494 to 6096867 Compare March 25, 2026 00:56
@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 25, 2026

@amehsamuel200225-sketch Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@MissBlue00
Copy link
Copy Markdown
Owner

@amehsamuel200225-sketch the ci/cd checks failed and there are merge conflicts, please resolve

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.

Auth: Implement merchant registration and authentication

2 participants