389-ds-base

Clone
Source Code
GIT

#51243 Issue 51229 - Server Settings page gets into an unresponsive loading state forever
Closed 3 years ago by spichugi. Opened 3 years ago by spichugi.
spichugi/389-ds-base i51229  into  master

file modified
+1 -1 
@@ -35,7 +35,7 @@ 
 

          this.selectNode = this.selectNode.bind(this);
 

      }
 

  

-     componentDidMount() {
 

+     componentDidUpdate() {
 

          if (this.props.wasActiveList.includes(1)) {
 

              if (this.state.firstLoad) {
 

                  this.loadConfig();

Bug Description: If we switch the tab in Cockpit UI and restart the instance -
we can't go back to the default tab as it's in an unresponsive loading state.

Fix Description: Do the update on ComponentDidUpdate instead of ComponentDidMount.

https://pagure.io/389-ds-base/issue/51229

Reviewed by: ?

Ack

rebased onto d1bfd69
3 years ago

rebased onto d1d557a
3 years ago

Pull-Request has been merged by spichugi
3 years ago

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/4296

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi
3 years ago
Metadata
None
No Tags
Changes Summary 1
+1 -1
file changed
src/cockpit/389-console/src/server.jsx
Powered by Pagure 5.13.3
Documentation • File an Issue • About • SSH Hostkey/Fingerprint
© Red Hat, Inc. and others.