Generator-jhipster: Update react dependencies with major version changes

Created on 27 Apr 2020 · 10Comments · Source: jhipster/generator-jhipster

Update the following dependencies in React implementation. It could be bit of work since we haven't updated these in a while

 react-toastify          4.5.2  →  5.5.0 
 react-transition-group  2.7.0  →  4.3.0 
 reactstrap              7.1.0  →  8.4.1 
 uuid                    3.3.3  →  7.0.3 
 html-webpack-plugin     3.2.0  →  4.2.0

$$ bug-bounty $$ $200 area react

Source

deepu105

Most helpful comment

I created a similar issue for Angular.

mraible on 27 Apr 2020

👍2

All 10 comments

@deepu105 if you're not already doing it I'm willing to give it a go, I have the same use case in one of my apps :)

MathieuAA on 27 Apr 2020

👍2

I created a similar issue for Angular.

mraible on 27 Apr 2020

👍2

@MathieuAA plz go ahead, I just did #11653 but the dependencies here needs bit more work. AFAIK react-toastify, react-transition-group and html-webpack-plugin has breaking changes. I suggest you branch off from #11653 since it has everything else uptodate

deepu105 on 27 Apr 2020

👍1

@deepu105 alright then

MathieuAA on 27 Apr 2020

First issue I've encountered, with the html-webpack-plugin dependency.

TypeError: Cannot read property 'tapAsync' of undefined
    at /home/mathieu/projects/munchy/node_modules/base-href-webpack-plugin/build/index.js:28:65
    ...

Found this issue in the repo that seems to correspond: https://github.com/jantimon/html-webpack-plugin/issues/1377#issuecomment-605678041
The base-href-webpack-plugin dependency didn't receive updates since early 2019. I'm gonna focus on the others 4 that seem to be easier to deal with.
I'll go back to this dependency after the others.

MathieuAA on 27 Apr 2020

I think this one is not very important anyway

On Mon, 27 Apr 2020, 8:54 pm Mathieu ABOU-AICHI, notifications@github.com
wrote:

First issue I've encountered, with the html-webpack-plugin dependency.

TypeError: Cannot read property 'tapAsync' of undefined
at /home/mathieu/projects/munchy/node_modules/base-href-webpack-plugin/build/index.js:28:65
...

Found this issue in the repo that seems to correspond: jantimon/html-webpack-plugin#1377
(comment)
https://github.com/jantimon/html-webpack-plugin/issues/1377#issuecomment-605678041
The base-href-webpack-plugin dependency didn't receive updates since
early 2019. I'm gonna focus on the others 4 that seem to be easier to deal
with.
I'll go back to this dependency after the others.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jhipster/generator-jhipster/issues/11651#issuecomment-620168735,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIOKF2W2FL2EUHW7Q5A773ROXIFNANCNFSM4MSCFLHA
.

deepu105 on 27 Apr 2020

👍1

So far, except the last dependency, everything's okay. There's just reactstrap, but it's very minor: https://github.com/reactstrap/reactstrap/blob/master/CHANGELOG.md#breaking-changes. I'm doing some more tests in real apps but I think you can include them in your PR

MathieuAA on 27 Apr 2020

👍1

BTW, I didn't see any use of the uuid package in the front-end code (except in the package.json file). Unless I'm missing something...

MathieuAA on 28 Apr 2020

👍1

Probably its from some legacy stuff or used only in some specific option
combination

On Tue, 28 Apr 2020, 12:43 am Mathieu ABOU-AICHI, notifications@github.com
wrote:

BTW, I didn't see any use of the uuid package in the front-end code
(except in the package.json file). Unless I'm missing something...

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jhipster/generator-jhipster/issues/11651#issuecomment-620273806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIOKF3IFVXO33KJGKYS4XTROYDA5ANCNFSM4MSCFLHA
.