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

fix: card-mod not working #247

Merged
merged 5 commits into from
Oct 3, 2023
Merged

fix: card-mod not working #247

merged 5 commits into from
Oct 3, 2023

Conversation

Gh61
Copy link
Contributor

@Gh61 Gh61 commented Jul 23, 2023

Hey,

I've created this pull request to fix the functionality issue with the card-mod as mentioned in issue #234.

I've added style and card_mod option properties. These are required to consist in config object for card-mod to read them and create styles.

Please review and provide feedback.
And thank you for your amazing work.

@Gh61 Gh61 mentioned this pull request Oct 2, 2023
src/config.ts Outdated Show resolved Hide resolved
@Gh61
Copy link
Contributor Author

Gh61 commented Oct 3, 2023

So I pushed changes to pass the remaining properties of config using the spread operator and I've tested it and it's working.

Copy link
Owner

@denysdovhan denysdovhan left a comment

Choose a reason for hiding this comment

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

Good. Please address a small note and let's merge it.

src/config.ts Outdated Show resolved Hide resolved
@Gh61
Copy link
Contributor Author

Gh61 commented Oct 3, 2023

You're totally right. Thank you for code reviewing, this operator is completely new for me.

@Gh61 Gh61 requested a review from denysdovhan October 3, 2023 19:27
Copy link
Owner

@denysdovhan denysdovhan left a comment

Choose a reason for hiding this comment

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

Good, thank you! Can to please also make this change to vacuum-card?

@denysdovhan denysdovhan merged commit 4df2497 into denysdovhan:master Oct 3, 2023
github-actions bot pushed a commit that referenced this pull request Oct 3, 2023
## [2.5.1](v2.5.0...v2.5.1) (2023-10-03)

### Bug Fixes

* card-mod not working ([#247](#247)) ([4df2497](4df2497))
@github-actions
Copy link

github-actions bot commented Oct 3, 2023

🎉 This PR is included in version 2.5.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

2 participants