update React v17

[youchann]

現状はできない。

• 依存関係のreact-selectがまだpeerDependencies内でv17を扱っていない(他のライブラリもそうである可能性がある)

• ライブラリreact-transition-group内で利用しているdeprecatedなメソッドReactDom.findDOMNode()をReact17で利用するとエラーが起こる
  ※以下は React17に上げる #200 内のexample/typescript内で利用した時のもの
  

[youchann]

facebook/react#20131

この辺りのissueが解決される。あるいは、react-transition-groupがよしなに対応してくれるのを待つ。

ちなみにreact-selectも内部でreact-transition-groupを利用している。

[youchann]

reactjs/react-transition-group#678
react-transition-group自体もReact17未対応っぽいな。

[youchann]

mui/material-ui#23311

17自体はほぼ未破壊な変更なので、peerDependenciesに17を盛り込みさえすれば、利用側でReact17を扱える。(ingred-ui自体に17を取り込まなくても対応は可能)

[youchann]

ちょっとやってみる

[youchann]

これらをみた感じだとyarn linkだと起こりうるエラーっぽい。
書いてあったことをしたらローカルでもデバッグできた

https://tech-1natsu.hatenablog.com/entry/2019/01/10/193043
facebook/react#14317

[youchann]

ただ利用側で本当に動くのか??というのがリリースしないとわからないのアレだな。。。
smart-hrさんはこのためのe2eテストを導入しているのかね..?

[youchann]

一応動作確認はできた。がすごいアナログ感なので何か良い方法を考えたい。

• ブランチ指定で社内プロダクトにyarn addする
• dist/(成果物)は生成されていないので、手元のingred-ui/distとすり替える
• 社内プロダクトで検証

みたいな感じで動作確認をした。

yarn add (ブランチ指定のingred-ui)を叩いた時点で依存関係は保証されるので、複数のReactが入るみたいなことはリリースしても起こらないはず。