chore: fix missing source-maps & reduce bundle size #781
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
Earlier in #772 , I have disabled source-maps temporarily for suppressing the warnings in the console.
Problem Statement
But that disabled source-maps overall for the entire gardens project and it is a bit hard to debug UI issues since it will point to the transpiled code.
Solution
This PR fixes the missing source-maps warnings and enables source-maps which would allow us for easy debugging over-all.
Also, apart from that, I installed
fork-bn.js
and aliased all thebn.js
chunks in the bundle to thefork-bn.js
. This allowed us to trim off ~46kB from the main bundle.Note:
Please follow the below mentioned steps for testing:
node_modules
&build
folders:rm -rf node_modules build
yarn install
oryarn
Images
Before:
After: