Lazy loading and client side updates

COS301-SE-2024 / The-Republic

Imagine a fusion of ‘Eskom se Push’ and ‘X’. The Republic allows users to raise concerns and spread awareness about government services through incident reports, with cool data visualizations providing a dynamic overview of public sentiment on service delivery.

https://the-republic-six.vercel.app

7 stars 0 forks source link

Lazy loading and client side updates #205

Closed IRIA7 closed 1 month ago

IRIA7 commented 1 month ago

Closes #169 :

Issues are now loaded page at a time as you scroll
The same for comments and replies
The page size can be set to anything over zero
The next page is preloaded when you scroll past the last fetched page

Closes #170

Adding, deleting and resolving issues now happens without a page reload
Adding and deleting comments also works without a reload

Also:

Fixed bug with deleting images when deleting posts
Updated styling for nav and reactions
Changed sing up button to login

IRIA7 commented 1 month ago

Yeah. Thanks bro, you can merge

hlokomani commented 1 month ago

Wait I am seeing that some react query stuff is gone, can @TebogoYungMercykay just double check the PR

IRIA7 commented 1 month ago

You're right. All react query calls in Feed and CommentList are removed since fetching is handled by LazyList now. We will have to add react query to LazyList later, there's a message on Discord about how we could do that. Also useMutate calls are commented out since they need to be changed to work with client side updating

TebogoYungMercykay commented 1 month ago

You're right. All react query calls in Feed and CommentList are removed since fetching is handled by LazyList now. We will have to add react query to LazyList later, there's a message on Discord about how we could do that. Also useMutate calls are commented out since they need to be changed to work with client side updating

I think we can work with that, We shall integrate react-query sometime later because we should use it only when we need it.