[Question] ControllerProviderは不要では？ - Githubissues

dash-app / dash-home

Home "Dashboard", Dash-Home.

MIT License

1 stars 0 forks source link

[Question] ControllerProviderは不要では？ #27

Closed Laica-Lunasys closed 3 years ago

Laica-Lunasys commented 3 years ago

Desc.

Provider(useContext)を利用せずにControllerコンポーネント自体でfetchすればよいのでは？
→ 親コンポーネントでのみ変更が実施される要素なので不必要

じゃあControllerProviderを撤廃するべき？

子コンポーネントの階層が大したことない(親から子へpropsで受け渡すので十分)なら、不必要。
~~しかし、AirconやLightなどの送信時といった、SummonPanelのようなSummonerを経てレンダリングされている場合、propsでの受け渡しだと複雑になりがちなため、Provider側で必要に応じて提供するのがベストそう? ~~ → 個別に実装しても大したことなかった

Laica-Lunasys commented 3 years ago

Removed https://github.com/dash-app/dash-home/commit/b51472359d105f330e64b861860367e00e9c6bc3