Closed trailstem closed 1 month ago
Latest commit: 39c43a56548aa2eb17522b93eabe08780f3d3fc0
The changes in this PR will be included in the next version bump.
Not sure what this means? Click here to learn what changesets are.
Click here if you're a maintainer who wants to add another changeset to this PR
Name | Link |
---|---|
Latest commit | 39c43a56548aa2eb17522b93eabe08780f3d3fc0 |
Latest deploy log | https://app.netlify.com/sites/ingred-ui/deploys/66a378c9f829120008bdc626 |
Deploy Preview | https://deploy-preview-1655--ingred-ui.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
@trailstem これはどのURLから動きを確認すればいいでしょうか?
@noronaoki FYI @deatiger
@trailstem これはどのURLから動きを確認すればいいでしょうか?
本件の修正内容は、Strorybook上, MultipleFilter
単体で動作確認することができないので、コードベースでレビューいただきたいですmm
(ingred-uiに関する変更だったため念の為 @noronaoki さんReviewerに入れさせていただきましたmm)
@trailstem なるほど、テーブルとの連動が正常にできるようになっていればLGTMです!
ingred-ui DataTable
とMultipleFilter
を組み合わせたExample作成する
修正背景
DataTable
と組み合わせて使用する際に、MultipleFilter
で一度選択した条件を編集した際にstateが更新されておらず、DataTable
に使用するデータをフィルタリングできないことがあったため修正Check List (If️ you added new component in this PR)
src/components/index.ts
.storybook/documents/Information/Samples/Samples.stories.tsx