diff options
author | Tristan Cacqueray <tdecacqu@redhat.com> | 2020-05-21 19:46:53 +0000 |
---|---|---|
committer | Tristan Cacqueray <tdecacqu@redhat.com> | 2020-05-21 20:21:59 +0000 |
commit | ff0e878865aa35580744527bd3558d40d28a9c88 (patch) | |
tree | 9c8dc32cc55cff2f8cdbbc223538cbd1f996fb7a | |
parent | 5350525e65ac6a6e848865480c45ccccaeb74a5e (diff) | |
download | zuul-ff0e878865aa35580744527bd3558d40d28a9c88.tar.gz |
web: remove status page componentDidUpdate implementation
It is not necessary to loadState when the timezone change as
this is already managed by react engine.
Moreover this method introduced a bug by shadowing the parent
Refreshable, which is in charge of fetching status page when
the tenant change.
Change-Id: I9fd35446c7fe98ce4bbb992983ec1ab56f3d32c5
-rw-r--r-- | web/src/pages/Status.jsx | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/web/src/pages/Status.jsx b/web/src/pages/Status.jsx index 84555b5d2..53a80f660 100644 --- a/web/src/pages/Status.jsx +++ b/web/src/pages/Status.jsx @@ -113,12 +113,6 @@ class StatusPage extends Refreshable { this.visibilityChangeEvent, this.visibilityListener) } - componentDidUpdate (prevProps) { - if (this.props.timezone !== prevProps.timezo) { - this.loadState() - } - } - setFilter = (filter) => { this.filter.value = filter this.setState({filter: filter}) |