Skip to content
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

The production build is broken due to the refactoring changes - Closes #4320 #4327

Merged
merged 4 commits into from
May 25, 2022

Conversation

reyraa
Copy link
Contributor

@reyraa reyraa commented May 25, 2022

What was the problem?

This PR resolves #4320

How was it solved?

Fixed the production build setup, as well as the issue with the package lock file.

How was it tested?

  • Installing dependencies using npm ci should work.
  • Installing dependencies using npm i should work.
  • You should be able run the app in dev mode: npm run dev.
  • You should be able to build the app in production using npm run build and then run it using a local server. for example http-server ./app/build.
  • You should be able to create distribution assets using npm run pack, or npm run pack:win.

@reyraa reyraa changed the base branch from development to feature/blockchain-agnostic-new May 25, 2022 10:22
@reyraa reyraa self-assigned this May 25, 2022
@reyraa reyraa force-pushed the 4320-fix-production-build branch from 6521866 to 20d33bc Compare May 25, 2022 10:25
@reyraa reyraa force-pushed the 4320-fix-production-build branch from 01031a1 to 4c1dfa9 Compare May 25, 2022 10:53
ikem-legend
ikem-legend previously approved these changes May 25, 2022
Copy link
Member

@ikem-legend ikem-legend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job!!!

app/package.json Show resolved Hide resolved
"postcss-for": "=2.1.1",
"postcss-functions": "3.0.0",
"postcss-loader": "3.0.0",
"postcss-mixins": "6.2.1",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why some deps version are lowered?

@ikem-legend ikem-legend self-requested a review May 25, 2022 15:14
@reyraa reyraa merged commit 22c0ae6 into feature/blockchain-agnostic-new May 25, 2022
@reyraa reyraa deleted the 4320-fix-production-build branch May 25, 2022 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

The production build is broken due to the refactoring changes
3 participants