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

Table の Header に Props の型を追加 #547

Merged
merged 5 commits into from
Oct 21, 2021

Conversation

takurinton
Copy link
Contributor

@takurinton takurinton commented Oct 21, 2021

以下のようなエラーが出ていたので修正。
ついでに react-select のバージョンを上げた。

error TS2559: Type '{ children: Element; }' has no properties in common with type 'IntrinsicAttributes & RefAttributes<HTMLTableSectionElement>'.

@netlify
Copy link

netlify bot commented Oct 21, 2021

✔️ Deploy Preview for ingred-ui ready!

🔨 Explore the source changes: 54f946f

🔍 Inspect the deploy log: https://app.netlify.com/sites/ingred-ui/deploys/61711ec61ae24a0007574640

😎 Browse the preview: https://deploy-preview-547--ingred-ui.netlify.app

Copy link
Contributor

@youchann youchann left a comment

Choose a reason for hiding this comment

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

react-selectのリリースノートなりをみて、問題なさそうなら:+1:

@takurinton
Copy link
Contributor Author

takurinton commented Oct 21, 2021

ここの対応をしました。 
react-select v5.1.0 で type の path の変更があったのでそれを適用しました。
https://react-select.com/upgrade
image

@youchann
Copy link
Contributor

#547 (review)

これみておいていただけると:pray:

@takurinton
Copy link
Contributor Author

よく見たらテストが落ちてた。。。

@takurinton
Copy link
Contributor Author

修正した。

@takurinton
Copy link
Contributor Author

すみません!さっき余計な変更があったので修正しました🙇‍♂️
また、リリースノートは、v5.1.0 で型の import の path が react-select/src から eact-select/dist/declarations/src に変更になったようなので適用しました。

@takurinton takurinton merged commit 278b9cc into voyagegroup:master Oct 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants