Closed hiroto0222 closed 2 years ago
@hiroto0222 ローカルで実行してみて特に気になる部分はなかったです!いいと思います!
細かいですが、他のページでリロードした時でもSettingsページに行ってしまったり、もしユーザがゲーム中に他のページに遷移したり、ゲームページ内でリロードした時の挙動も余裕があれば今後確認していきたいですね。
次のIssueを出すのでマージしちゃいます!
レビューありがとうございます!
細かいですが、他のページでリロードした時でもSettingsページに行ってしまったり、もしユーザがゲーム中に他のページに遷移したり、ゲームページ内でリロードした時の挙動も余裕があれば今後確認していきたいですね。
現在 onAuth アクション内でログインしている場合 SettingsPage へ遷移するという処理を行っていますが、login アクション内でその処理を行った方が良いですね。
Issue: #1
目的
達成条件
実装の概要
Firebase の初期化
Firebase 認証
要認証ナビゲーションガード / Routes
name, meta: { requiresAuth: true || false }
を追加レビューして欲しいところ
不安に思っていること・質問