[issue-19] Firebaseの権限エラーを回避するためデッキ詳細ページのgetStaticPropsを無効化

TBSten commented 1 year ago

目的・解決すること

close #19

非公開のデッキの詳細ページに作成者本人がアクセスしようとしてもエラーが発生してアクセスできない件の暫定対処を実施。

変更内容

デッキ詳細ページでのSGを無効化

スクリーンショット

デザインの変更なし

テスト項目

[ ] スマートフォンでも動作するか確認した（影響ないと考えられる場合は省略可）
- ログインのドメインの関係で検証不可能なのでスキップ
[x] UI の変更の場合、画面サイズを変えてもレイアウトが崩れないか確認した

チェック項目

[x] 適切なレビュワーをアサインした。

備考

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
zutomayo-card-web	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 22, 2023 3:46am

Mishima-Ryuji commented 1 year ago

@TBSten 確認して良い状態になったらレビュワーの指定をお願いします。

Mishima-Ryuji commented 1 year ago

@TBSten Firebase Authの承認済みドメインに192.168.xxxを追加すればスマホでもログインできると思います。

Mishima-Ryuji commented 1 year ago

@TBSten getStaticPropsの中でサーバー側からpermission deniedが返ってきたら、何も返さないようにしてフロント側だけで、データ取得を行ったほうがいいような気がします。これだと、全てのデッキで読み込む速度が遅くなってしまうことが気になります。お手数ですが、ご対応いただけますと幸いです。

TBSten commented 1 year ago

@TBSten getStaticPropsの中でサーバー側からpermission deniedが返ってきたら、何も返さないようにしてフロント側だけで、データ取得を行ったほうがいいような気がします。これだと、全てのデッキで読み込む速度が遅くなってしまうことが気になります。お手数ですが、ご対応いただけますと幸いです。

たしかに、権限ないときだけにしたほうがいいですね！対応しますー

TBSten commented 1 year ago

@TBSten getStaticPropsの中でサーバー側からpermission deniedが返ってきたら、何も返さないようにしてフロント側だけで、データ取得を行ったほうがいいような気がします。これだと、全てのデッキで読み込む速度が遅くなってしまうことが気になります。お手数ですが、ご対応いただけますと幸いです。

以下コミットで対応しました！また、getStaticPropsでcardsがサーバ側で取得できた場合はclient側でそれらを利用できた方がコスト的に良さそうなので ⚡️ サーバ側でエラーが発生しても取得した値が破棄されないように修正で途中でエラーが発生してもすでに取得できてある分はclientに返すようにしました。

Mishima-Ryuji / ZutomayoCard