From 8977948b4077054ac7709d1685632d11360b8584 Mon Sep 17 00:00:00 2001
From: Ryan Lerch <rlerch@redhat.com>
Date: Dec 06 2017 04:30:37 +0000
Subject: Remove some options from user config dialog


* remove the avatar option in userhub config #470
* remove hub visibility option from userhub config #484
* remove user management tabs from userhubs config #483
* comment out the other tab that said it wasnt implemented #485

Signed-off-by: Ryan Lerch <rlerch@redhat.com>

---

diff --git a/hubs/static/client/app/components/HubConfig/HubConfigDialog.js b/hubs/static/client/app/components/HubConfig/HubConfigDialog.js
index 6d00e05..3f42ed6 100644
--- a/hubs/static/client/app/components/HubConfig/HubConfigDialog.js
+++ b/hubs/static/client/app/components/HubConfig/HubConfigDialog.js
@@ -93,34 +93,41 @@ export default class HubConfigDialog extends React.Component {
             globalConfig={this.props.globalConfig}
             tabTitle={<FormattedMessage {...messages.general} />}
             handleChange={this.props.onConfigChange}
+            hub={this.props.hub}
             />
-          <UserPanel
-            users={this.props.users.owner}
-            globalConfig={this.props.globalConfig}
-            role="owner"
-            handleChange={this.props.onUserChange}
-            tabTitle={<FormattedMessage {...messages.owners} />}
-            urls={this.props.urls}
-            currentUser={this.props.currentUser}
-            />
-          <UserPanel
-            users={this.props.users.member}
-            globalConfig={this.props.globalConfig}
-            role="member"
-            handleChange={this.props.onUserChange}
-            tabTitle={<FormattedMessage {...messages.members} />}
-            urls={this.props.urls}
-            currentUser={this.props.currentUser}
-            />
-          <ChatPanel
-            hubConfig={this.props.hubConfig}
-            globalConfig={this.props.globalConfig}
-            tabTitle={<FormattedMessage {...messages.chat} />}
-            handleChange={this.props.onConfigChange}
-            />
-          <NotImplementedPanel
+          {!this.props.hub.user_hub &&
+            <UserPanel
+              users={this.props.users.owner}
+              globalConfig={this.props.globalConfig}
+              role="owner"
+              handleChange={this.props.onUserChange}
+              tabTitle={<FormattedMessage {...messages.owners} />}
+              urls={this.props.urls}
+              currentUser={this.props.currentUser}
+              />
+          }
+          {!this.props.hub.user_hub &&
+            <UserPanel
+              users={this.props.users.member}
+              globalConfig={this.props.globalConfig}
+              role="member"
+              handleChange={this.props.onUserChange}
+              tabTitle={<FormattedMessage {...messages.members} />}
+              urls={this.props.urls}
+              currentUser={this.props.currentUser}
+              />
+          }
+          {!this.props.hub.user_hub &&
+            <ChatPanel
+              hubConfig={this.props.hubConfig}
+              globalConfig={this.props.globalConfig}
+              tabTitle={<FormattedMessage {...messages.chat} />}
+              handleChange={this.props.onConfigChange}
+              />
+          }
+          {/*<NotImplementedPanel
             tabTitle={<FormattedMessage {...messages.other} />}
-            />
+            />*/}
         </TabSet>
       </Modal>
     );
diff --git a/hubs/static/client/app/components/HubConfig/HubConfigPanelGeneral.js b/hubs/static/client/app/components/HubConfig/HubConfigPanelGeneral.js
index 00b1e44..cce20a5 100644
--- a/hubs/static/client/app/components/HubConfig/HubConfigPanelGeneral.js
+++ b/hubs/static/client/app/components/HubConfig/HubConfigPanelGeneral.js
@@ -100,6 +100,7 @@ export default class GeneralPanel extends React.Component {
               />
           </p>
         </div>
+        {!this.props.hub.user_hub &&
         <div className="form-group row">
           <label htmlFor="hub-settings-general-visibility">
             <FormattedMessage {...messages.visibility} />
@@ -120,6 +121,8 @@ export default class GeneralPanel extends React.Component {
             <FormattedHTMLMessage {...messages.visibility_help} />
           </p>
         </div>
+        }
+        {!this.props.hub.user_hub &&
         <div className="form-group row">
           <label htmlFor="hub-settings-general-avatar">
             <FormattedMessage {...messages.avatar} />
@@ -135,6 +138,7 @@ export default class GeneralPanel extends React.Component {
             <FormattedMessage {...messages.avatar_help} />
           </p>
         </div>
+        }
       </form>
     );
   }
diff --git a/hubs/static/client/app/components/HubConfig/index.js b/hubs/static/client/app/components/HubConfig/index.js
index 8446004..4a4062b 100644
--- a/hubs/static/client/app/components/HubConfig/index.js
+++ b/hubs/static/client/app/components/HubConfig/index.js
@@ -101,6 +101,7 @@ class HubConfig extends React.Component {
             onUserChange={this.doUserChange}
             urls={this.props.urls}
             currentUser={this.props.currentUser}
+            hub={this.props.hub}
             />
         }
       </div>
diff --git a/hubs/static/client/app/components/TabSet.js b/hubs/static/client/app/components/TabSet.js
index 25dad18..b601b93 100644
--- a/hubs/static/client/app/components/TabSet.js
+++ b/hubs/static/client/app/components/TabSet.js
@@ -20,6 +20,7 @@ export default class TabSet extends React.Component {
 
   render() {
     var tabLinks = this.props.children.map(function(panel, index) {
+      if (!panel){return null;}
       return (
         <Tab active={this.state.activeTab === index}
              linkClicked={(e) => this.linkClicked(e, index)}
@@ -30,6 +31,7 @@ export default class TabSet extends React.Component {
     }.bind(this));
 
     var tabPanels = this.props.children.map(function(panel, index) {
+      if (!panel){return null;}
       return (
         <TabPanel
           active={this.state.activeTab === index}