Skip to content

Conversation

laminne
Copy link
Member

@laminne laminne commented Sep 20, 2025

close #381

概要

  • メールアドレスでログインできるようにしました.
  • バックエンド側が完成したらdraftを解除します.

@laminne laminne requested a review from Copilot September 20, 2025 16:21
@laminne laminne self-assigned this Sep 20, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enables login functionality using email addresses instead of account names, updating both the frontend form and API integration to support email-based authentication.

  • Changed login form input from account name to email address with appropriate validation
  • Updated API function to accept email parameter instead of name parameter
  • Modified form labels and placeholders to reflect email-based login

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
app/routes/login.tsx Updated login form to use email input field and modified action handler to process email instead of name
app/lib/api/login.ts Changed LoginArgs interface and login function to use email parameter instead of name

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@laminne laminne marked this pull request as ready for review September 30, 2025 16:16
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@laminne laminne requested a review from MikuroXina October 2, 2025 14:21
@laminne laminne merged commit a290d13 into main Oct 3, 2025
1 check passed
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.

fix: メールアドレスでログイン出来るようにする
2 participants