-
-
Notifications
You must be signed in to change notification settings - Fork 65
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
Updates for Dash 3.0 #506
Updates for Dash 3.0 #506
Conversation
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.
💃 Looks great. I'll note that there are a couple of places (Checkbox
and a couple of charts are the ones I noticed) where you omitted a default value of false
that was previously explicit. This shouldn't change the behavior at all, it'll just change the docstring, but to my eye leaving these implicit is intuitive.
Replacement for PR 499.
Please see #499 for more details and sample apps
Closes #453
This PR replaces #458 which only handled changes for Dash 3. This PR includes updates for Dash 3 that are also compatible with Dash 2.
_dashprivate_loadingState
used in theSkeleton
component.Checkbox
component which has an icon component prop that needs to be rerender with additional props._dashprivate_layout
: Checkbox, Popover, Menu, Hovercard, Timeline, Stepperpackage.json
, pin "dash-extensions-js": "0.0.8" . Before upgrading, check for Dash 2 compatibility. This package is required for the renderDashComponent function, which may be affected by updates in future Dash versions.Note - do not use the new typing support because it does not handle Mantine typing correctly.
ignore this comment in the build and add notes to releasing.md to ensure it's not added in the future