-
Notifications
You must be signed in to change notification settings - Fork 16
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
SmartNodeSelector
: Extended property state updates
#151
SmartNodeSelector
: Extended property state updates
#151
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.
Some minor suggestions only to consider - otherwise LGTM.
let hasError = false; | ||
let error = ""; |
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.
let hasError = false; | |
let error = ""; | |
let error; |
CHANGELOG.md
Outdated
- [#140](https://github.com/equinor/webviz-core-components/pull/140) - Improved styling of the `Select` component. | ||
- [#145](https://github.com/equinor/webviz-core-components/pull/145) - Added wrapper components for typically used Dash components (Dropdown, Slider, etc) with additional styling. | ||
- [#148](https://github.com/equinor/webviz-core-components/pull/148) - Changed default value of `numSecondsUntilSuggestionsAreShown` to 0.5 in `SmartNodeSelector` component. | ||
- [#151](https://github.com/equinor/webviz-core-components/pull/151) - `SmartNodeSelector`: Changes to `data` and `delimiter` props are considered now and cause the component to update. |
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.
There were quite some changes to this file. Change to only adding the 151-entry, and instead consider if we want to enforce further style rules on CHANGELOG.md
in separate issue/PR?
}); | ||
} catch (e) { | ||
this.treeData = null; | ||
hasError = true; |
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.
hasError = true; |
hasError: hasError, | ||
error: error, |
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.
hasError: hasError, | |
error: error, | |
hasError: error !== undefined, | |
error: error || "", |
Changes to `data` and `delimiter` are considered now.
8178c01
to
c55287b
Compare
Adjusted changelog format
Changes to
data
anddelimiter
are considered now and cause the component to update.