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.

Sign up for GitHub

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

Jump to bottom

feat: ログインができるように #11

Merged
merged 4 commits into from
Oct 8, 2024
Merged

feat: ログインができるように #11

merged 4 commits into from
Oct 8, 2024

Conversation

laminne
Copy link
Member

@laminne laminne commented Oct 8, 2024 •
edited
Loading

やったこと

close #7

  • 最低限のスタイリング
  • ログインページを実装
    • APIのリクエスト/トークンの保存
    • refresh_tokenは保存していません(APIが未実装なので保存しても使えない)

@laminne laminne added the T: feature New feature label Oct 8, 2024
@laminne laminne self-assigned this Oct 8, 2024
MikuroXina
MikuroXina requested changes Oct 8, 2024
View reviewed changes
app/hooks/login.ts Outdated Show resolved Hide resolved
app/root.module.css Outdated Show resolved Hide resolved
app/root.module.css Outdated Show resolved Hide resolved
app/root.module.css Show resolved Hide resolved
MikuroXina
MikuroXina requested changes Oct 8, 2024
View reviewed changes
app/routes/login.tsx Outdated Show resolved Hide resolved
@laminne laminne merged commit d1764fc into main Oct 8, 2024
1 check passed
@laminne laminne deleted the feat/7-login branch October 8, 2024 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MikuroXina MikuroXina MikuroXina approved these changes

@Allianaab2m Allianaab2m Awaiting requested review from Allianaab2m

@nanai10a nanai10a Awaiting requested review from nanai10a

@m1sk9 m1sk9 Awaiting requested review from m1sk9

Assignees

@laminne laminne

Labels
T: feature New feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

feat: ログインができるようにする
2 participants
@laminne @MikuroXina