-
Notifications
You must be signed in to change notification settings - Fork 0
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
💄 354 removed watcher from filedropzone and reformated #355
💄 354 removed watcher from filedropzone and reformated #355
Conversation
I removed the |
And i moved the default story to the top - it should be the first story to be seen. |
…gs-without-watcher' into 354-add-example-to-delete-warnings-without-watcher
As this would be a breaking change for users of this component I would like to get @markostreich to review this, too. To me it looks good. I don't know if I like the automatic approach of the watcher or the more functional approach more. Code wise it LGTM |
Yes would be good if @markostreich looks over this one! |
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested the new approach successfully. We can adopt the change. As mentioned in my two comments, the presentation in the Storybook should now also be adjusted to include a demonstration of the clearance of warnings so that interested users can see how it works.
I am unfortunately no longer part of this project. |
huh o.O |
🎉 This PR is included in version 2.1.0-beta.6 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 2.1.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
Reference
Issues #354