From d3928a0c0f6dcf7be7f5d76ff6d0992f589c2138 Mon Sep 17 00:00:00 2001
From: mabashian <mabashia@redhat.com>
Date: Wed, 18 Nov 2020 11:34:54 -0500
Subject: [PATCH] Get rid of onExpand but keep isExpanded on side nav

---
 .../src/components/AppContainer/NavExpandableGroup.jsx | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/awx/ui_next/src/components/AppContainer/NavExpandableGroup.jsx b/awx/ui_next/src/components/AppContainer/NavExpandableGroup.jsx
index 9122a97272..45c3b91fae 100644
--- a/awx/ui_next/src/components/AppContainer/NavExpandableGroup.jsx
+++ b/awx/ui_next/src/components/AppContainer/NavExpandableGroup.jsx
@@ -7,12 +7,10 @@ class NavExpandableGroup extends Component {
   constructor(props) {
     super(props);
     const { routes } = this.props;
-    this.state = { isExpanded: true };
 
     // Extract a list of paths from the route params and store them for later. This creates
     // an array of url paths associated with any NavItem component rendered by this component.
     this.navItemPaths = routes.map(({ path }) => path);
-    this.handleExpand = this.handleExpand.bind(this);
     this.isActiveGroup = this.isActiveGroup.bind(this);
     this.isActivePath = this.isActivePath.bind(this);
   }
@@ -26,13 +24,8 @@ class NavExpandableGroup extends Component {
     return Boolean(matchPath(history.location.pathname, { path }));
   }
 
-  handleExpand(e, isExpanded) {
-    this.setState({ isExpanded });
-  }
-
   render() {
     const { groupId, groupTitle, routes } = this.props;
-    const { isExpanded } = this.state;
 
     if (routes.length === 1) {
       const [{ path }] = routes;
@@ -46,10 +39,9 @@ class NavExpandableGroup extends Component {
     return (
       <NavExpandable
         isActive={this.isActiveGroup()}
-        isExpanded={isExpanded}
+        isExpanded
         groupId={groupId}
         title={groupTitle}
-        onExpand={this.handleExpand}
       >
         {routes.map(({ path, title }) => (
           <NavItem