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.

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

購入報告一覧ページの isDisabled のスタイルが正常に当たらない問題を解決 #522

Merged
merged 9 commits into from
Apr 9, 2023

Conversation

YushiroDodo63
Copy link
Contributor

対応Issue

resolve #492

概要

購入報告ページの編集・削除ボタンの isDisabeld やそのスタイルが正常に当たらない問題があったので、それを解決しました。

原因

おそらく、Recoil で管理している user を使っていることが原因でした。
Recoil で管理している値はレンダリング後に保持される値で、SSR時に仮想DOMを作成した時は userundefined なので編集・削除ボタンは isDisabledfalse になるが、CSR時には user が存在しているので、その部分で問題が発生していた。(場合によっては Hydration Error が出たりしていた。)

解決方法

user を使用した条件分岐を行う際は、手間だが currentUser を取得するAPIを叩き、取得した currentUser を用いて条件分岐を行うことで解決できる。

画面スクリーンショット等

2023-04-09.16.06.26.mov

テスト項目

  • http://localhost:3000/purchasereports にて新規購入報告を作成し、作成した購入報告の編集・削除できるか
  • 作成した購入報告の編集・削除ボタンのスタイルが上記動画と同じように active になっているか

備考

@YushiroDodo63 YushiroDodo63 self-assigned this Apr 9, 2023
@github-actions github-actions bot added the bug Something isn't working label Apr 9, 2023
@YushiroDodo63
Copy link
Contributor Author

学内募金の動作

2023-04-09.20.38.06.mov

購入申請の動作

2023-04-09.20.35.12.mov

Copy link
Collaborator

@imaimai17468 imaimai17468 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
コード、動作ともに問題なしです!

@imaimai17468 imaimai17468 merged commit 18c3e1a into develop Apr 9, 2023
@imaimai17468 imaimai17468 deleted the fix/dodo/492-switch-isDiabled-by-bureau branch April 9, 2023 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BureauによってDisabledが切り替えられるようにする
2 participants