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
Implement React lazy loading and Upgrade to webpack 4 #1164
base: develop
Are you sure you want to change the base?
Conversation
Rebasing.
"start": "NODE_ENV=development node server", | ||
"precommit": "pretty-quick --staged", | ||
"test": "jest", | ||
"test:watch": "npm test -- --watch" | ||
"test:watch": "npm test -- --watch", |
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.
want to use yarn vs npm for consistency?
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.
Sure I will change it to yarn.
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.
Can you also fix the merge conflict by rebasing?
The Python tests have some problem where it does not run correctly. Did you run the node tests to make sure nothing broke?
Sure will rebase and fix the conflict. |
The initial load time for the web interface is very large because of the large js file. This is a result of the complete static components loading on the first page itself. My PR aims to resolve the speed issue by removing dead code from the production build of the Webpack and implementing react lazy loading.
Description
Performance comparison before and after using Chrome lighthouse:
Before
After
Motivation and Context
A website experience is defined by how smooth it can function and how fast it can function also this is a step forward in upgrading the project with the latest dependencies.
Reviewers
@viyatb @sharmamohit123
Types of changes
Checklist: