staking-launchpad icon indicating copy to clipboard operation
staking-launchpad copied to clipboard

Bump web3.js to v1.7.5

Open hwwhww opened this issue 2 years ago • 3 comments

To fix the recent Brave browser issue, this PR bumps web3.js package version to v1.7.5.

TODO:

  • [x] Test deposit function

hwwhww avatar Aug 24 '22 08:08 hwwhww

Hey @samajammin @wackerow This PR works for me locally. But somehow Netlify didn't install successfully.

Newbie question: I clicked the "Re-run" button of one failed check and then saw "You have successfully requested Header rules - serene-carson-3331d5 be rerun": image

But I couldn't find the re-run job. Any idea of how to resolve it?

hwwhww avatar Aug 24 '22 08:08 hwwhww

@hwwhww This appears to be a Node JS version issue...

1:18:32 AM: error [email protected]: The engine "node" is incompatible with this module. Expected version ">=14.16". Got "10.24.1"
1:18:32 AM: error Found incompatible module.

We appear to be using 10.24.1 (lts/dubnium) while this is requiring >=14.16 (lts/fermium).

@CarlBeek Have we looked into updating the node version in the past? Any idea if that would affect anything else on the site? I generally use lts/gallium -> v16.17.0 as my default, and locally the site has always worked, and this PR also works fine if using that version, but breaks as soon as I switch to 10.24.1

wackerow avatar Aug 24 '22 17:08 wackerow

In general, updating the packages would be good. I made an attempt in the past to update everything to the latest stable versions and then lost a day to dependency hell trying to figure out what was and wasn't working. That said, this is probably more a fault of my JS skills than anything else.

CarlBeek avatar Aug 24 '22 17:08 CarlBeek

@wackerow I rebased this PR with #546 changes, it works now! 🎉

hwwhww avatar Sep 02 '22 06:09 hwwhww

Guys please merge it as soon as possible

meshin-dev avatar Sep 07 '22 12:09 meshin-dev

What is the status of this?

meshin-dev avatar Sep 14 '22 12:09 meshin-dev