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.

Sign up for GitHub

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

Jump to bottom

tarot.json - update tarot card data #1

Open
michaeljoelt opened this issue Jun 10, 2022 · 3 comments
Open

tarot.json - update tarot card data #1

michaeljoelt opened this issue Jun 10, 2022 · 3 comments
Labels
documentation Improvements or additions to documentation help wanted Extra attention is needed

Comments

@michaeljoelt
Copy link
Owner

I just borrowed someone elses free-to-use code to get started with the tarot data. I have noticed mistakes, typos, duplicate info, etc. I'll start tracking them here as I notice specifics, but was also just hoping that someone familiar with tarot may be down to start working on updating this. :D <3
@michaeljoelt michaeljoelt added help wanted Extra attention is needed documentation Improvements or additions to documentation labels Jun 10, 2022
@michaeljoelt
Copy link
Owner Author

not necessarily "documentation", but this issue is definitely more focused on general knowledge of tarot, not coding. please help if you're knowledgeable.

@michaeljoelt michaeljoelt changed the title JSON - update tarot card data main.js / tarot.json - update tarot card data Jun 10, 2022
@VeitchM
Copy link
Contributor

VeitchM commented Jun 12, 2022

It has been done

@michaeljoelt
Copy link
Owner Author

hey - thanks for your help! I think you meant to comment on this issue: #2
I merged your changes and closed the related issue. Thanks for your help with this!!

@michaeljoelt michaeljoelt changed the title main.js / tarot.json - update tarot card data tarot.json - update tarot card data Jun 12, 2022
michaeljoelt pushed a commit that referenced this issue Jun 20, 2022
@ashinsabu
Merge pull request #1 from michaeljoelt/main
91cd9f8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@michaeljoelt @VeitchM