Qin-Eva / pj_engivia

0 stars 1 forks source link

Closed kijikunnn closed 3 years ago

kijikunnn commented 3 years ago

34 の対応

stateとして，ユーザーIDのみを取得しています．
"/"からログインページに遷移したり，ログアウトをするという前提で"/"にstateを作成しています．
ひとまず，検証も兼ねて"/"と"/posts"の2つのページでstateを読み込んでいますが，今後必要なページ・コンポーネントにstateを読み込んでいきます．
firebase.tsを編集する際にwarningが出ており，ドキュメントを見る限りだとv8の記法のようだったのでv9の記法に修正しました． https://firebase.google.com/docs/auth/web/start?hl=ja

kijikunnn commented 3 years ago

@yugo-ibuki 確認ありがとうございます！各レビューの該当行削除しました！それと同時に，firestore.tsで型に対するwarningが多く出たので，そちらも対応しました．

yugo-ibuki commented 3 years ago

ありがとうございます！助かります！

yugo-ibuki commented 3 years ago

@kijikunnn コンフリクトだけ解消しました！ちなみに、今の感じだとindexページのみに反映されているぽいので _app.tsxとかで読み込んで、配下のコンポーネントに浸透させるのがいいかも？

kijikunnn commented 3 years ago

@yugo-ibuki Layout.tsxにstateを移動しました！それと同時に初期値を””からnullに変更しました．

yugo-ibuki commented 3 years ago

ありがとうございます！明日確認します！

yugo-ibuki commented 3 years ago