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

FilterIssue #150

Closed himanshujasuja1040 closed 3 years ago

himanshujasuja1040 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

Someone is attempting to deploy a commit to a Personal Account owned by @stephin007 on Vercel.

@stephin007 first needs to authorize it.

himanshujasuja1040 commented 3 years ago

@Justinnn07

stephin007 commented 3 years ago

Complete the PR template @himanshujasuja1040

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/eQZNGgydSNmDyDgDHAHyFpbLRtgV
✅ Preview: https://cowin-vaccine-git-fork-himanshujasuja1040-himanshujasuj-d018b3.vercel.app

stephin007 commented 3 years ago

@himanshujasuja1040 Add no data found component, when no data is found according to the added filters.

Justinnn07 commented 3 years ago

@himanshujasuja1040 great work! Few points to be noted:-

  1. Please remove the pagination ..
  2. If the filtered array is 0, please show the no data found component ..
  3. The Fare filter is not working ..

Thanks

Justinnn07 commented 3 years ago

@himanshujasuja1040 any update?

stephin007 commented 3 years ago

Outdated PR, closing it!