Commit 42aaa2b9 by Torkel Ödegaard

redux: improved state handling

parent 3fd707f3
...@@ -41,14 +41,20 @@ export class AlertRuleList extends PureComponent<Props, State> { ...@@ -41,14 +41,20 @@ export class AlertRuleList extends PureComponent<Props, State> {
} }
componentDidMount() { componentDidMount() {
this.fetchRules(this.getStateFilter()); console.log('did mount');
this.fetchRules();
}
componentDidUpdate(prevProps: Props) {
if (prevProps.stateFilter !== this.props.stateFilter) {
this.fetchRules();
}
} }
onStateFilterChanged = evt => { onStateFilterChanged = evt => {
this.props.updateLocation({ this.props.updateLocation({
query: { state: evt.target.value }, query: { state: evt.target.value },
}); });
this.fetchRules(evt.target.value);
}; };
getStateFilter(): string { getStateFilter(): string {
...@@ -59,8 +65,8 @@ export class AlertRuleList extends PureComponent<Props, State> { ...@@ -59,8 +65,8 @@ export class AlertRuleList extends PureComponent<Props, State> {
return 'all'; return 'all';
} }
async fetchRules(stateFilter: string) { async fetchRules() {
await this.props.getAlertRulesAsync({ state: stateFilter }); await this.props.getAlertRulesAsync({ state: this.getStateFilter() });
} }
onOpenHowTo = () => { onOpenHowTo = () => {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment