Use React.lazy for loading components

[antoinechalifour]

Type: Refactor

The following has been addressed in the PR: Use React.Lazy for loading components

• There is a related issue? Nop
• Unit or Functional tests are included in the PR None changed

[codecov]

Codecov Report

Merging #158 into master will increase coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
+ Coverage   83.15%   83.27%   +0.12%     
==========================================
  Files          98       97       -1     
  Lines         944      933      -11     
  Branches      165      148      -17     
==========================================
- Hits          785      777       -8     
- Misses        142      144       +2     
+ Partials       17       12       -5

Impacted Files	Coverage Δ
src/router.tsx	82.35% <100%> (ø)	⬆️
src/components/DetailSidebar/styles.ts	100% <0%> (ø)	⬆️
src/components/Header/Header.tsx	89.47% <0%> (ø)	⬆️
src/components/Install/Install.tsx	83.33% <0%> (ø)	⬆️
...mponents/RegistryInfoDialog/RegistryInfoDialog.tsx	66.66% <0%> (ø)	⬆️
src/components/DetailSidebar/DetailSidebar.tsx	89.47% <0%> (+4.47%)	⬆️
src/components/Dist/Dist.tsx	100% <0%> (+11.11%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f1f8f8a...e4f494e. Read the comment docs.

[juanpicado]

Could you share any analysis that prove the impact of this changes in the bundle size? chunks etc etc ? master vs this PR

[antoinechalifour]

I'll try to give you some numbers tomorrow but here's the short version :

• using React.lazy is the idiomatic way of doing code splitting in react land
• it removed the asyncComponent file so the bundle should be lighter

[priscilawebdev]

@antoinechalifour Well done! 👏 Maybe you could also remove the file "asynComponent", because it is not being used anymore

[antoinechalifour]

I think it's already been removed ! I'll check tomorrow

[juanpicado]

@antoinechalifour thanks for bring the router and code splitting to the next level 👏