status-im/fathom
2
0
Fork 0
mirror of https://github.com/status-im/fathom.git synced 2025-02-28 19:10:36 +00:00
Code Issues Projects Releases Wiki Activity

delay initial request until datepicker component bubbled up the correct period

Browse Source
This commit is contained in:
Danny 2018-05-09 10:55:56 +02:00
parent e73a205d26
commit 3c823e9351
3 changed files with 7 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
assets/js/components/CountWidget.js
View File

@ -24,10 +24,6 @@ class CountWidget extends Component {
} }
} }
componentDidMount() {
this.fetchData();
}
componentWillReceiveProps(newProps, prevState) { componentWillReceiveProps(newProps, prevState) {
if(newProps.before == prevState.before && newProps.after == prevState.after) { if(newProps.before == prevState.before && newProps.after == prevState.after) {
return; return;

2
assets/js/components/DatePicker.js
View File

@ -31,9 +31,7 @@ class DatePicker extends Component {
before: 0, before: 0,
after: 0, after: 0,
} }
}
componentDidMount() {
this.setTimeRange(this.state.period) this.setTimeRange(this.state.period)
} }

11
assets/js/components/Table.js
View File

@ -21,10 +21,6 @@ class Table extends Component {
} }
} }
componentDidMount() {
this.fetchRecords()
}
componentWillReceiveProps(newProps, prevState) { componentWillReceiveProps(newProps, prevState) {
if(newProps.before == prevState.before && newProps.after == prevState.after) { if(newProps.before == prevState.before && newProps.after == prevState.after) {
return; return;
@ -40,9 +36,16 @@ class Table extends Component {
@bind @bind
fetchRecords() { fetchRecords() {
this.setState({ loading: true }); this.setState({ loading: true });
let before = this.state.before;
let after = this.state.after;
Client.request(`${this.props.endpoint}?before=${this.state.before}&after=${this.state.after}&limit=${this.state.limit}`) Client.request(`${this.props.endpoint}?before=${this.state.before}&after=${this.state.after}&limit=${this.state.limit}`)
.then((d) => { .then((d) => {
// request finished; check if timestamp range is still the one user wants to see
if( this.state.before != before || this.state.after != after ) {
return;
}
this.setState({ this.setState({
loading: false, loading: false,
records: d, records: d,