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

Migrate dual validated range #59689

Merged
merged 29 commits into from
Mar 16, 2020
Merged
Changes from 1 commit
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
9713e71
Move validated range files to new NP location
Mar 9, 2020
c53f20d
Update refs in code
Mar 9, 2020
1cc4646
Clean up old validated range files
Mar 9, 2020
c4daf99
Change relative paths to 'kibana-react'. Some clean up
Mar 9, 2020
48d0eaa
Change to relative paths
Mar 9, 2020
14728d9
Merge branch 'master' into np-migrate-dual-validated-range
elasticmachine Mar 9, 2020
6a5f8a9
Fix i18n errors
Mar 9, 2020
3c0cac4
i18n clean up. Export module explicitly
Mar 10, 2020
8a5dd9d
Change files over to TS to prevent build issue where validated range …
Mar 10, 2020
61d2c7d
Merge remote-tracking branch 'upstream/master' into np-migrate-dual-v…
Mar 10, 2020
4c6f1b5
Clean up TS conversion
Mar 10, 2020
90550b8
More clean up. Extend EuiRangeProps
Mar 11, 2020
a761969
Merge remote-tracking branch 'upstream/master' into np-migrate-dual-v…
Mar 11, 2020
18e05ae
Remove unneeded ts-ignore
Mar 11, 2020
c4961c2
Merge branch 'master' into np-migrate-dual-validated-range
elasticmachine Mar 11, 2020
326de0d
Review feedback and test fixes
Mar 12, 2020
787d93a
Merge remote-tracking branch 'upstream/master' into np-migrate-dual-v…
Mar 12, 2020
f51f1d5
Merge branch 'np-migrate-dual-validated-range' of github.com:aaronjca…
Mar 12, 2020
66ab9d1
Change double to single quotes
Mar 12, 2020
96ab294
min and max aren't always passed, make optional
Mar 12, 2020
daa1361
Type updates
Mar 12, 2020
2324691
Merge remote-tracking branch 'upstream/master' into np-migrate-dual-v…
Mar 12, 2020
eafe4b9
Merge remote-tracking branch 'upstream/master' into np-migrate-dual-v…
Mar 13, 2020
13f4d45
Review feedback. Set state to empty on init and add ignore comment
Mar 13, 2020
a51fc45
Review feedback
Mar 13, 2020
c5acf30
Merge remote-tracking branch 'upstream/master' into np-migrate-dual-v…
Mar 13, 2020
dabe3b9
Add back in last 2 ts-ignores. Build fails without focusable attribut…
Mar 13, 2020
85b1887
Merge branch 'master' into np-migrate-dual-validated-range
elasticmachine Mar 16, 2020
2e92b23
Rollback change to state init. Initializing state to null actually tr…
Mar 16, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Review feedback
Aaron Caldwell committed Mar 13, 2020
commit a51fc45782da7b460f220cce3b8317ac12331676
Original file line number Diff line number Diff line change
@@ -68,8 +68,7 @@ export class ValidatedDualRange extends Component<Props> {
return null;
}

// @ts-ignore state populated by getDerivedStateFromProps
state: State = {};
state!: State;

_onChange = (value: Value) => {
const { isValid, errorMessage } = isRangeValid(
@@ -99,7 +98,6 @@ export class ValidatedDualRange extends Component<Props> {
value, // eslint-disable-line no-unused-vars
onChange, // eslint-disable-line no-unused-vars
allowEmptyRange, // eslint-disable-line no-unused-vars
// @ts-ignore
...rest
} = this.props;

@@ -117,8 +115,6 @@ export class ValidatedDualRange extends Component<Props> {
fullWidth={fullWidth}
value={this.state.value}
onChange={this._onChange}
// @ts-ignore
focusable={false} // remove when #59039 is fixed
kindsun marked this conversation as resolved.
Show resolved Hide resolved
{...rest}
/>
</EuiFormRow>