stephin007 / Cowin-Vaccine-Availablity-Checker

CoWIN Vaccination Tracker, Below is the PRODUCTION LINK this is updated at end of each day. To see any latest Updates, please check the documentation
https://cowinvaccinetracker.forcommunity.tech/
MIT License
27 stars 19 forks source link

fixed loading issue #138

Closed Justinnn07 closed 3 years ago

Justinnn07 commented 3 years ago

🚨 Please review the guidelines for contributing to this repository.

Please check if the PR fulfills these requirements

Describe your changes

❤️ Thank you!

vercel[bot] commented 3 years ago

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/stephin007/cowin-vaccine-availablity-checker/97js1Uy3a7viacYvYMtq3wkiFp3k
✅ Preview: https://cowin-vaccine-git-justinnn07-bugloading-ste-1c8424.vercel.app

stephin007 commented 3 years ago

@Justinnn07 left few comments!

stephin007 commented 3 years ago

@Justinnn07 again left some review comments

stephin007 commented 3 years ago

@Justinnn07 why is the prop removed in line 22 of nullState.js?

Could you please explain!

Justinnn07 commented 3 years ago

@Justinnn07 why is the prop removed in line 22 of nullState.js?

Could you please explain!

the prop vaccineData is removed due to multiple passing of data from Home.js, which will replicate the vaccine info ..

Let me know if there is any other changes!