hey icon indicating copy to clipboard operation
hey copied to clipboard

fix: lazy load dm convo list

Open bhavya2611 opened this issue 2 years ago • 2 comments

What does this PR do?

Adds lazy loading to fetch the preview messages for the conversations list

Type of change

  • [ ] Enhancement (non-breaking small changes to existing functionality)

How should this be tested?

  • [ ] See if everything on messaging works correctly
  • [ ] If you have a lot of conversations see if the loading is quicker and smoother

bhavya2611 avatar Dec 05 '22 13:12 bhavya2611

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

Name Status Preview Updated
api ✅ Ready (Inspect) Visit Preview Dec 6, 2022 at 5:44AM (UTC)
pro ✅ Ready (Inspect) Visit Preview Dec 6, 2022 at 5:44AM (UTC)
web ✅ Ready (Inspect) Visit Preview Dec 6, 2022 at 5:44AM (UTC)

vercel[bot] avatar Dec 05 '22 13:12 vercel[bot]

Getting closer: It looks like it's the useListConversations hook that is the bottleneck right now. Going to investigate how many requests we're still making in there 👀

Screenshot 2022-12-05 at 5 21 12 PM

elisealix22 avatar Dec 05 '22 22:12 elisealix22

Closing this out for now as we continue to investigate! For more context on where we're at, see this comment: https://github.com/lensterxyz/lenster/pull/1366#issuecomment-1342726519

elisealix22 avatar Dec 08 '22 21:12 elisealix22