Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FE: Authentication Implementation #110

Open
wants to merge 5 commits into
base: staging
Choose a base branch
from

Conversation

atrincas
Copy link

@atrincas atrincas commented Jan 7, 2025

Authentication System Implementation

Key Changes

  • Authentication Service: Refactored into a class-based structure with dedicated methods for:

    • Sign in
    • Password reset request
    • Password change
    • Improved error handling
  • New Auth Pages:

    • Sign in (/auth/signin)
    • Forgot password (/auth/forgot-password)
    • Change password (/auth/change-password/[token])
    • Sign up with token (/auth/signup/[token])
  • UI Components:

    • Added AuthWrapper component for consistent auth page layouts
    • New button themes: 'transparent' and 'outline'
    • Made button theme and size props optional

Testing Instructions

  1. Test sign-in flow with valid and invalid credentials
  2. Verify forgot password flow:
    • Request password reset
    • Check email delivery
    • Reset password with token
  3. Test error states and validation messages
  4. Verify responsive layout on different screen sizes

Jira tickets

@atrincas atrincas added the front-end Front end label Jan 7, 2025
Copy link

vercel bot commented Jan 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fora-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 7, 2025 10:44am

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
front-end Front end
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant