Added a useQuery hook to fetch the events data from the server using axios.
Implemented a loading indicator using Material UI Skeletons while the data is being fetched.
Reorganized the code to make it more readable and maintainable.
Tested the implementation thoroughly to ensure that it works as expected.
I believe this updated code is a more efficient and maintainable solution compared to the previoususeEffect() implementation. Please review and merge the changes if they meet your requirements. You can find more details about the changes and the reference link to the original issue in the PR.
Yes, I agree with you @chrisvrose, this is still a POC, however we can improve many things, perhaps @dragonfist453, @MayurSChittaragi can initiate the discussion.
useQuery
hook to fetch the events data from the server using axios.I believe this updated code is a more efficient and maintainable solution compared to the previous
useEffect()
implementation. Please review and merge the changes if they meet your requirements. You can find more details about the changes and the reference link to the original issue in the PR.Reference : https://github.com/IEEE-RVCE/IEEE-RVCE.github.io/issues/99
Requesting a review from the OG author(s) of the code as they are the ones who wrote it.
Please Note: This implementation is more like a prototype