-
Notifications
You must be signed in to change notification settings - Fork 208
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
fix: sha3 package build error #722
Conversation
response to build error from indirect sha3 package dependency
✅ Deploy Preview for dapper-rolypoly-9814ad ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I've built this into my changes in #721 but we can merge it in isolation to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Always love a commit that removes 10x the lines it adds, LGTM!
README.md
Outdated
@@ -26,7 +26,7 @@ The Infura Project ID will enable the network status progress bar showing the ba | |||
REACT_APP_INFURA_PROJECT_ID=your-infura-project-id-here | |||
``` | |||
|
|||
Note: The Portis wallet option is not available when running locally | |||
Note: The Portis wallet option is currently not available |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we even really need to have this here, I think we can just nuke Portis & note something about removing it in a changelog & commit history
sha3
indirect dependency tracing to@web3-react/portis-connector
package no longer compatible, causing builds to break during dependency installation. removing package while still on older version of node to restore buildsAlso patches a linting error from recent Nethermind page change, removing unused imports.