Skip to content
This repository has been archived by the owner on Jul 5, 2022. It is now read-only.

Display the # of COVID cases on a 3D map #2821

Merged
merged 5 commits into from
Oct 29, 2020

Conversation

JacobGeoGeek
Copy link
Contributor

This PR is related to #2615 in which I have displayed number of cases on a 3D map by using the height property of the boxes

Happy Hacktoberfest 2020

Community Contribution

Link to live project, video, or image:

https://editor.p5js.org/JacobGeoGeek/full/RjkCQVEU8

Sharing

  • I would be happy to have my project shared on The Coding Train's social media!

this PR is related to CodingTrain#2615 in which I have displayed number of cases on a 3D map by using the `height` property of the boxes 

Happy Hacktoberfest 2020
Copy link

@featurepeek-bot featurepeek-bot left a comment

Choose a reason for hiding this comment

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

This pull request has been automatically deployed to FeaturePeek. ✨
Your deployment will be kept up-to-date with this pull request's latest changes.

https://peek.run/lpmhe69p


Please read our docs for more configuration details.

@JacobGeoGeek
Copy link
Contributor Author

Hi @gruselhaus and @violetcraze

Recently there were some changes on the main branch. So, I merged the main branch into my branch. however, the Jest / Jest Unite Tests failed for this reason: Cannot read property 'user' of undefined

Do you guys have any idea of what's the cause of the problem?
Cheers.

@JacobGeoGeek
Copy link
Contributor Author

JacobGeoGeek commented Oct 28, 2020

Just fixe the problem (remove the extra space at the end)

Copy link
Member

@duskvirkus duskvirkus left a comment

Choose a reason for hiding this comment

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

Looks good! Glad you figured out why the unit testing failed. Thanks for sharing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants