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

add sign in page #203

Merged
merged 5 commits into from
Nov 13, 2024
Merged

add sign in page #203

merged 5 commits into from
Nov 13, 2024

Conversation

iibarbari
Copy link
Collaborator

@iibarbari iibarbari commented Nov 8, 2024

This pull request includes significant changes to both the frontend and backend of the application. The most important changes involve the addition of a new SignIn component, updates to various CSS files for styling consistency, and a minor update to an error message in the backend.

Frontend Changes:

  • New SignIn Component:

    • Added SignIn component with form handling and layout integration (frontend/src/components/SignIn/index.tsx).
    • Created FormTitle sub-component for the SignIn form (frontend/src/components/SignIn/FormTitle/index.tsx).
    • Added CSS modules for SignIn and FormTitle components (frontend/src/components/SignIn/SignIn.module.css, frontend/src/components/SignIn/FormTitle/FormTitle.module.css) [1] [2].
  • CSS Refactoring:

    • Reorganized and added new CSS files for various components like buttons, forms, alerts, and containers (frontend/src/styles/components/button.css, frontend/src/styles/components/form.css, frontend/src/styles/components/alert.css, frontend/src/styles/components/container.css) [1] [2] [3] [4].
    • Updated existing components to use the new layout component (frontend/src/components/Error/index.tsx, frontend/src/components/Home/index.tsx) [1] [2].
  • Helper Functions:

    • Added a new helper function for base64 encoding (frontend/src/helpers/base64.ts).

Backend Changes:

  • Error Message Update:
    • Changed the unauthorized error message from "You are not authorized." to "Invalid credentials." in the Responder component (backend/pkg/restapi/responder/main.go).

Copy link
Owner

@th0th th0th left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💐

frontend/src/components/App/index.tsx Outdated Show resolved Hide resolved
frontend/src/components/SignIn/index.tsx Outdated Show resolved Hide resolved
frontend/src/components/SignIn/index.tsx Outdated Show resolved Hide resolved
frontend/src/styles/components/alert.css Outdated Show resolved Hide resolved
frontend/src/styles/components/spinner.css Outdated Show resolved Hide resolved
Copy link
Owner

@th0th th0th left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@iibarbari iibarbari merged commit f77e744 into reloaded Nov 13, 2024
3 checks passed
@iibarbari iibarbari deleted the sign-in branch November 13, 2024 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants