react-timeseries-charts icon indicating copy to clipboard operation
react-timeseries-charts copied to clipboard

Refresh Typescript version in `updates-typescript`

Open igozali opened this issue 6 years ago • 2 comments

So I took some time to refresh the Typescript version (3.4.5) on the long-awaited updates-typescript branch in the hopes that it would have better compatibility with storybook (kind of a long shot since I have no idea yet how storybook works).

I got it to a point where I can run lerna bootstrap && lerna build and only have errors in website_ts and website (i.e. react-axis and react-timeseries-charts compiles fine). Since I don't quite know which one is being worked on more, I decided to stop here and ask for your comments.

Quick summary:

  • Update typescript to 3.4.5. There were quite a lot of downstream issues as a result of this update (mostly incompatibilities in typing), but luckily I didn't have to change much of the code to make it work, and most of the changes are in package.json file.
  • Use [email protected] since react-scripts-ts is deprecated
  • Changed a few dependencies in website_ts to make React typings work (previously it was failing with this error)
  • Did a bit of work for typings in website_ts.

Hope it helps a bit, and would be great if you could try it out. Let me know if you have any comments or questions. :)

igozali avatar May 03 '19 06:05 igozali

In case you don't have notifications enabled.. @pjm17971 @sartaj10

igozali avatar May 04 '19 00:05 igozali

Thanks for this, I'm planning to try it out.

pjm17971 avatar May 08 '19 20:05 pjm17971