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

Refactor code to dynamically generate themes and cards #107

Closed

Conversation

MaximilianoCabreraP
Copy link

@MaximilianoCabreraP MaximilianoCabreraP commented Oct 17, 2024

I've made some changes to simplify how we handle themes and cards in the project. Instead of manually adding each card one by one, the code now imports them dynamically. This means any new card added to the ./src/cards directory gets picked up automatically—no extra coding required!

Olso i update the README.md.

closes #108

@akanshSirohi
Copy link
Owner

@MaximilianoCabreraP This is nice update, but it will change the route of every card, you have to rename each file in this PR to keep the route intact.

@MaximilianoCabreraP
Copy link
Author

It's not necessary; the route runs fine and adapts everything. Here is the URL where I'm running it on my localhost for you to review: https://rna-vegas-closing-senators.trycloudflare.com
image

@MaximilianoCabreraP
Copy link
Author

i saw the issue, my bad 🙏
Im working on it

@akanshSirohi
Copy link
Owner

@MaximilianoCabreraP I will close this PR in 24 hours or please make the changes.

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

Successfully merging this pull request may close these issues.

[REFACTOR] Dinamically get available cards
2 participants