flash-calendar icon indicating copy to clipboard operation
flash-calendar copied to clipboard

[feat]: add support for a callback for activeDayFiller

Open isaiasdearcog opened this issue 1 year ago • 9 comments

Addresses this request: https://github.com/MarceloPrado/flash-calendar/issues/76

isaiasdearcog avatar Dec 21 '24 04:12 isaiasdearcog

🦋 Changeset detected

Latest commit: 6883b40984fb096e0456f15d71f759fed41bceaf

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@marceloterreiro/flash-calendar Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

changeset-bot[bot] avatar Dec 21 '24 04:12 changeset-bot[bot]

Looks great! Can you also add a story to test the functionality?

MarceloPrado avatar Dec 25 '24 10:12 MarceloPrado

I actually saw you suggested this should be able at any level on theming, which makes a lot of sense, and I would like to work on that contribution! I'll see if I can work on that over the weekend :). Can you please show me an example of tests for components so I can build test suites for this feature? When I worked on this I wanted to add the test but I couldn't find any component tests.

isaiasdearcog avatar Dec 26 '24 19:12 isaiasdearcog

@MarceloPrado friendly reminder of the comment above!

isaiasdearcog avatar Jan 10 '25 14:01 isaiasdearcog

Hey @MarceloPrado, can we move forward with this PR? Would love to include it in my next release! Thank you.

hugoh59 avatar Jan 16 '25 18:01 hugoh59

I wish I could approve this PR!

hugoh59 avatar Jan 29 '25 23:01 hugoh59

@MarceloPrado i see the package has been gaining some popularity lately! If you are interested on supporting this, let me know :)

isaiasdearcog avatar Mar 12 '25 20:03 isaiasdearcog

@MarceloPrado 👀

Pamavoc avatar Mar 22 '25 09:03 Pamavoc

hi folks, sorry! I need to improve my GH notifications setup.

@isaiasdearcog for testing, I'm currently using storybook - no snapshot testing yet, just a story dedicated to this feature that we can run and explore if things are working. In the future, we could use something like Chromatic to snapshot each story.

For the purpose of this PR, it would be good to see a few stories testing the feature. You can attach a video of your simulator going over them

MarceloPrado avatar Mar 28 '25 19:03 MarceloPrado

@isaiasdearcog added the missing test here: 9f2d5f1

https://github.com/user-attachments/assets/6e2b5f99-f0c8-4b89-b3a6-dadb8a87a71e

Will merge this PR and publish the next version. Thanks for your contribution!

MarceloPrado avatar Nov 24 '25 13:11 MarceloPrado