-
Notifications
You must be signed in to change notification settings - Fork 2
Recoil を Zustand に置き換えるリファクタリング #1038
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
base: develop
Are you sure you want to change the base?
Conversation
Summary of ChangesHello @TkymHrt, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! このプルリクエストは、将来のReact 19へのアップデートを見据え、現在の状態管理ライブラリであるRecoilをZustandにリファクタリングするものです。主要な状態(認証情報やユーザーデータ)の管理をZustandストアに移行し、永続化機能もZustandのミドルウェアで再実装しました。これにより、アプリケーションの基盤がよりモダンで将来性のあるものになります。 Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code Review
このプルリクエストは、RecoilからZustandへの状態管理ライブラリの移行を目的としており、React 19への将来的な対応を見据えた素晴らしいリファクタリングです。全体として、移行はきれいに実行されており、Zustandのpersistミドルウェアの扱いや、ストアの構成も適切です。
1点、MainLayoutコンポーネントのuseEffectフックの依存配列について、認証状態の変更に即座に反応するために修正を提案しました。ご確認いただければ幸いです。
view/next-project/src/components/layout/MainLayout/MainLayout.tsx
Outdated
Show resolved
Hide resolved
Deploying finansu with
|
| Latest commit: |
0645514
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://f8d24b71.finansu.pages.dev |
| Branch Preview URL: | https://refactor-yama-migrate-to-zus.finansu.pages.dev |
対応Issue
https://nut-m-e-g.slack.com/archives/C020WQ3GY07/p1768051810911529
概要
Recoil を Zustand に置き換えるリファクタリングを行いました!
React 19 において Recoil が動作しないため、アプデを見越してのPRです。
主な変更点は以下です。
authAtom/userAtom→authStore/userStoreに移行RecoilRootを削除pnpmに統一画面スクリーンショット等
テスト項目
ビルド・Lint
make buildが成功することmake run-lintが通ること認証フロー
ページ遷移と権限制御
/my_page,/purchase_report_list,/fund_informations等)が以前通り動作すること備考
ついでに権限ないユーザーがページにアクセスしたとき、内容が一瞬見れてた問題も解消しておきました✌