implement lazy load on contributions section in contributor's profile page

coronasafe / leaderboard

Leaderboard collects data from GitHub and Slack to show off the work of our open source contributors

https://contributors.ohc.network/

MIT License

13 stars 46 forks source link

implement lazy load on contributions section in contributor's profile page #416

Closed Harshal141 closed 2 months ago

Harshal141 commented 2 months ago

OHC Network Contributors - 31 March 2024 - Watch Video

This is the final product for the issue https://github.com/coronasafe/leaderboard/issues/415

vercel[bot] commented 2 months ago

@Harshal141 is attempting to deploy a commit to the Open Healthcare Network Team on Vercel.

A member of the Team first needs to authorize it.

Harshal141 commented 2 months ago

Ok I will send the performance diffe.
When I ran the lint-fix command these files checkData.js and loadOrgData.js were updated. There are no code changes just ' is replaced by " and spacing is uniform.
Will remove the merge conflict.
ok I will update the title to implementing lazy load on contributions section in contributor's profile page

Harshal141 commented 2 months ago

These are the performance analysis I got before(main) and after(download)

@rithviknishad

download.pdf

main.pdf

vercel[bot] commented 2 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
leaderboard	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 4, 2024 3:34am

rithviknishad commented 2 months ago

@Harshal141 ensure prettier is configured...

Harshal141 commented 2 months ago

@rithviknishad I ran this command,

prettier ./components/contributors/GithubActivity.tsx --write

Harshal141 commented 2 months ago

Can i get a review on this

rithviknishad commented 2 months ago

All of the data is still being delivered to the client... How would this reduce the page size?

Harshal141 commented 2 months ago

There were changes regarding this profile page which are on prod now. Let's update this PR with the latest commit and if it still doesn't show any improvement we can drop this idea.