Skip to content

PR to fix build issues, so that we can deploy to Netlify - #3

Open
KohliSuraj wants to merge 5 commits into
developfrom
bug/fix-build-issue
Open

PR to fix build issues, so that we can deploy to Netlify#3
KohliSuraj wants to merge 5 commits into
developfrom
bug/fix-build-issue

Conversation

@KohliSuraj

Copy link
Copy Markdown
Member
  • migrating to pnpm instead of yarn
  • correct node version for project is v14.21.1
  • fixing import issues

TODO: Fix error. After wallet connect, emp-tools gives an unauthorised error. This error is coming from bnc-onboard and bnc-notify packages of blocknative.

@netlify

netlify Bot commented Jan 31, 2023

Copy link
Copy Markdown

Deploy Preview for sumero-emp-tools failed.

Name Link
🔨 Latest commit e5fed76
🔍 Latest deploy log https://app.netlify.com/sites/sumero-emp-tools/deploys/63d909aab8b36400075aa947

@KohliSuraj

KohliSuraj commented Feb 1, 2023

Copy link
Copy Markdown
Member Author

FYI Build is working fine with node v14.21.1 and pnpm 6.31.0 locally. Netlify apparently only supports pnpm for node v16.14.0 and above. Have raised a support query with them.

https://app.netlify.com/sites/sumero-emp-tools/deploys/63d909aab8b36400075aa947

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.

1 participant