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

Add tarot cards and Ouija board #73752

Merged
merged 8 commits into from
May 15, 2024

Conversation

aapocalypseAArisen
Copy link
Contributor

@aapocalypseAArisen aapocalypseAArisen commented May 13, 2024

Summary

Content "Added tarot cards as a variant of playing cards and added Ouija board as a playable board game."

Purpose of change

I noticed a lack of mystically inclined games and decided to add them.

Describe the solution

Added these games into Cataclysm.

Describe alternatives you've considered

Not having these games. Nothing horrible would happen but it would be an opportunity not taken.

Testing

Used the debug menu to spawn in both the tarot cards and Ouija board. Successfully played them like any other game.

Additional context

First time contributing anything. We'll see how it goes.

Adds tarot cards as a variant of playing cards.
Adds Ouija board to list of board games.
Adds tarot cards as a variant of the deck of cards.
Adds Ouija board to pool of board games.
Fix hard crash
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON new contributor labels May 13, 2024
@aapocalypseAArisen aapocalypseAArisen changed the title Testbranch Add tarot cards and Ouija board May 13, 2024
data/json/items/fluff.json Outdated Show resolved Hide resolved
data/json/items/fluff.json Outdated Show resolved Hide resolved
aapocalypseAArisen and others added 2 commits May 13, 2024 16:58
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
data/json/items/fluff.json Outdated Show resolved Hide resolved
data/json/items/fluff.json Outdated Show resolved Hide resolved
data/json/items/fluff.json Outdated Show resolved Hide resolved
aapocalypseAArisen and others added 3 commits May 13, 2024 22:37
Co-authored-by: Jianxiang Wang (王健翔) <qrox@sina.com>
Co-authored-by: Jianxiang Wang (王健翔) <qrox@sina.com>
Co-authored-by: Jianxiang Wang (王健翔) <qrox@sina.com>
@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels May 14, 2024
@PGR-14
Copy link

PGR-14 commented May 14, 2024

Maybe we could add something in w/ portal storms?

@aapocalypseAArisen
Copy link
Contributor Author

aapocalypseAArisen commented May 15, 2024

Am I missing anything that would allow merging? I just want to make sure, I'm new at doing this.

@Maleclypse Maleclypse merged commit 7f06e9f into CleverRaven:master May 15, 2024
20 of 28 checks passed
@aapocalypseAArisen aapocalypseAArisen deleted the testbranch branch May 15, 2024 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions new contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants