Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added the 3rd option for showing per country covid data #175

Merged
merged 26 commits into from
Jun 28, 2021

Conversation

stephin007
Copy link
Owner

@stephin007 stephin007 commented Jun 25, 2021

🚨 Please review the guidelines for contributing to this repository.

Please check if the PR fulfills these requirements

  • Make sure you are requesting to NEW-UI. Don't request other protected Branches like staging/master
  • Make sure no conflicts are present in the code, if so please resolve it(Tip: Always fetch upstream)
  • Your Commit messages should make sense.
  • Don't push your package.lock.json as this project uses yarn.lock already.
  • Check your code additions will fail neither code linting checks nor unit test.

Describe your changes

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Ennhancement

  • What is the current behavior? (You can also link to an open issue here)
    Does not show country covid data.

  • What is the new behavior (if this is a feature change)?
    Added extra option in the CovidWorldData for showing country and further for each country data will be shown.

  • Does this PR introduce a breaking change? (i.e changes that may require users to update/refactor existing instances of the application)
    NO

  • Other Information:
    NONE

❤️ Thank you!

@stephin007 stephin007 added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed new feature up-for-grabs Yet to be assigned/"snatched" to/by someone labels Jun 25, 2021
@stephin007 stephin007 requested a review from Justinnn07 June 25, 2021 19:18
@stephin007 stephin007 self-assigned this Jun 25, 2021
@vercel
Copy link

vercel bot commented Jun 25, 2021

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/5TQC22TUt3MHh8B9ny84twtSRZZn
✅ Preview: https://cowin-vaccine-availablity-checker-git-enhance-150ca2-stephin007.vercel.app

@stephin007 stephin007 marked this pull request as draft June 25, 2021 19:38
@stephin007 stephin007 marked this pull request as ready for review June 28, 2021 12:06
@stephin007
Copy link
Owner Author

@Justinnn07 can you please review this , but dont merge just approve it, if everything is looking okay to be merged

@Justinnn07
Copy link
Collaborator

@Justinnn07 can you please review this , but dont merge just approve it, if everything is looking okay to be merged

Sure!

Copy link
Collaborator

@Justinnn07 Justinnn07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work @stephin007 💥💥

@stephin007 stephin007 merged commit 4cd8871 into NEW-UI Jun 28, 2021
stephin007 added a commit that referenced this pull request Jun 30, 2021
* Added Scroll to top fucntionality

* added firacode font CDN

* Added 3rd option in the World Covid Info. (#171)

* Added single continent info(WIP)

* Added continent options

* Added 3rd option and added horizontal scroll(WIP)

* removed width css and added text to tell user to swipe to see more data

* removed width css and added text to tell user to swipe to see more data

* added text to p tags

* fixed css for the continent Dropdown

* removed unwanted css

* removed whole continents data/graph

* added check for continent chart

* added conflicts workflow  (#173)

* added conflicts workflow 🔨

* added staging to the workflow 🔥

* changed the token name 🔨

* Added the 3rd option for showing per country covid data (#175)

* Added single continent info(WIP)

* Added continent options

* Added 3rd option and added horizontal scroll(WIP)

* removed width css and added text to tell user to swipe to see more data

* removed width css and added text to tell user to swipe to see more data

* added text to p tags

* fixed css for the continent Dropdown

* removed unwanted css

* removed whole continents data/graph

* added check for continent chart

* uncommented the third option

* updated chart content for continent view and removed dataset label

* Added country dropdown for each continent

* Added country flag  for dataReception check

* fixed img tag for vercel deployment error fix

* added the correct jmespath for th destrcutured flag

* added check for empty object for flags

* made continent and country single select responsive

* Added country card for information

* Added country card for information

* fixed css

* Added check for the country right card to be only shown when actual data is fetched

* finalised country card

Co-authored-by: Justin Varghese <69856008+Justinnn07@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed new feature up-for-grabs Yet to be assigned/"snatched" to/by someone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants