Skip to content

tarot-ai-reader#196

Merged
Shubhamsaboo merged 4 commits intoShubhamsaboo:mainfrom
maurizioorani:the-tarot-reader
Jun 15, 2025
Merged

tarot-ai-reader#196
Shubhamsaboo merged 4 commits intoShubhamsaboo:mainfrom
maurizioorani:the-tarot-reader

Conversation

@maurizioorani
Copy link
Contributor

LLM can now interpret tarot cards

LLM can now interpret tarot cards
@maurizioorani
Copy link
Contributor Author

I would like to contribute. Best regards.

@Shubhamsaboo
Copy link
Owner

Very interesting application. Thank you so much for your contribution.

Is it possible to make it in English for wider audience consumption so everyone can understand and use this app. For now it's a bit restrictive to Italian speaking audience.

@maurizioorani
Copy link
Contributor Author

I'm so pleased you find my application interesting. It would be an honor for me to be part of the great community you've created, and I have other interesting projects to share.

You are right: I've modified the application to make it more international, removing all references to Italian, using open source card images in English as well: I already pushed the modified code to the main branch. Thank you again, and congratulations on your work—your passion truly shines through!

@Shubhamsaboo
Copy link
Owner

Thank you so much for accepting my suggestions.

Here's one more -- Do you mind updating your README to reflect the latest code? It still references to the old Italian implementation. And instead of screenshots, can you record a screencast demo to show people how to use this app?

@maurizioorani
Copy link
Contributor Author

Nice, I updated the readme with a screencast explaining how to use the application. Thank you for your suggestion, I like it.

@Shubhamsaboo Shubhamsaboo merged commit 4f3a81c into Shubhamsaboo:main Jun 15, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants