Create account

[mana1127]

Closes ログインしていない場合は/accountにリダイレクトさせる

概要

/account/page.tsxにsrc\app\Components\UserForm\UserForm.tsxの中身を全て移す
ログインしていない場合はどのページに到達しても/loginにリダイレクトさせる
/accountでは各種ログインを選択できるようにする

変更内容

確認方法

チェックリスト

• PRに必要な内容を記述し、Assignやタイトルを設定した
• File Changedで不要な変更や誤った変更が無いか確認した
• 対応したissueをProjectの"レビュー中・待機中"に移動した
• レビューを頼んだ人にDiscordでお願いした
• 機密情報が含まれていないことを確認した(含まれている場合は直ちにリモートからコミットを削除すること)
• UIを変更した場合
  • PCで見た時に表示が崩れていないことを確認した
  • スマホでも正常に表示されることを確認した(大きい画面と小さい画面両方で)

[github-actions]

Visit the preview URL for this PR (updated for commit 37acd02):

https://todo-real-c28fa--pr47-create-account-p4fqk9dd.web.app

_{(expires Fri, 06 Dec 2024 04:59:53 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: cbf065b4d5e2f44e47cd594ff42f749fc108f410}

[MurakawaTakuya]

@mana1127
PRの先頭のClosesの部分にはissueの名前とリンクを入れてほしいです(以下のPR作成時のテンプレートの説明の通りです)

<!-- PRをmergeしたら自動でissueをcloseしたい場) -->
<!-- Closes [表示したい文字、issueタイトルがおすすめ](issueのURL) -->

<!-- PRにissueをリンクだけしたい場合(自動でcloseはしない) -->
<!-- Related to [表示したい文字、issueタイトルがおすすめ](issueのURL) -->

例えば今回であれば

Closes [ログインしていない場合は`/account`にリダイレクトさせる](https://github.com/MurakawaTakuya/todo-real/issues/38)

こんな感じです

[MurakawaTakuya]

ログインしていない場合はどのページに到達しても/loginにリダイレクトさせる
この実装を忘れてそうなので実装お願いします

参考までに
Functions: redirect
【Next.js】App Routerでリダイレクトする方法

これを使って、src\app\page.tsxのreturnの上とかでユーザーのログイン状況に応じてリダイレクトすればいいかと
ユーザー情報の参照方法は次のコードのようにするとapp以下のディレクトリで参照できます

todo-real/src/app/Components/UserForm/UserForm.tsx

Line 16 in 7b365b5

const { user } = useUser();

[MurakawaTakuya]

ログインしている場合はこのボタンも不要なので、こんな感じで演算子で囲っておきましょう

Suggested change

	<CenteredToggleButtonGroup
	value={formMode}
	exclusive
	onChange={handleFormModeChange}
	aria-label="form mode"
	>
	<ToggleButton value="register" aria-label="register">
	新規登録
	</ToggleButton>
	<ToggleButton value="login" aria-label="login">
	ログイン
	</ToggleButton>
	</CenteredToggleButtonGroup>
	{!user && (
	<CenteredToggleButtonGroup
	value={formMode}
	exclusive
	onChange={handleFormModeChange}
	aria-label="form mode"
	>
	<ToggleButton value="register" aria-label="register">
	新規登録
	</ToggleButton>
	<ToggleButton value="login" aria-label="login">
	ログイン
	</ToggleButton>
	</CenteredToggleButtonGroup>
	)}

[MurakawaTakuya]

#47 (comment)
これのテスト環境で/accountが404になるのがちょっと謎かも、様子見てこっちで対応しておきます
https://todo-real-c28fa--pr47-create-account-p4fqk9dd.web.app/account
SPAの問題とかかなぁ?

[MurakawaTakuya]

コメント漏れと一個だけ確認
src\app\Components\UserForm\UserForm.tsxも消していい気がしたけど残してるのって何か理由あったっけ?

[MurakawaTakuya]

たぶんこの辺のコメントも消してOKですね

[MurakawaTakuya]

@mana1127
#47 (comment) resolved 2688993
https://todo-real-c28fa--pr47-create-account-p4fqk9dd.web.app/account/

このコメントの修正を入れたので、作業をする前に

git pull origin create-account

をするようにお願いします

[MurakawaTakuya]

LGTM!