Merge pull request #10869 from AlexSCorey/10822-fixReactWarnings

Fixes several react warnings

awx/ui/src/components/Lookup/Lookup.js

@@ -89,6 +89,11 @@ function Lookup(props) {
   }, [state.selectedItems, multiple]);
 
   const clearQSParams = () => {
+    if (!history.location.search) {
+      // This prevents "Warning: Hash history cannot PUSH the same path;
+      // a new entry will not be added to the history stack" from appearing in the console.
+      return;
+    }
     const parts = history.location.search.replace(/^\?/, '').split('&');
     const ns = qsConfig.namespace;
     const otherParts = parts.filter((param) => !param.startsWith(`${ns}.`));

awx/ui/src/screens/ActivityStream/ActivityStream.js

@@ -256,8 +256,8 @@ function ActivityStream() {
             renderToolbar={(props) => (
               <DatalistToolbar {...props} qsConfig={QS_CONFIG} />
             )}
-            renderRow={(streamItem) => (
-              <ActivityStreamListItem streamItem={streamItem} />
+            renderRow={(streamItem, index) => (
+              <ActivityStreamListItem key={index} streamItem={streamItem} />
             )}
           />
         </Card>

awx/ui/src/screens/InstanceGroup/InstanceGroup.js

@@ -66,7 +66,7 @@ function InstanceGroup({ setBreadcrumb }) {
       name: (
         <>
           <CaretLeftIcon />
-          {t`Back to instance groups`}
+          {t`Back to Instance Groups`}
         </>
       ),
       link: '/instance_groups',

awx/ui/src/screens/InstanceGroup/InstanceGroup.test.js

@@ -32,7 +32,7 @@ describe('<InstanceGroup/>', () => {
 
   test('should render expected tabs', async () => {
     const expectedTabs = [
-      'Back to instance groups',
+      'Back to Instance Groups',
       'Details',
       'Instances',
       'Jobs',

awx/ui/src/screens/Organization/OrganizationDetail/OrganizationDetail.js

@@ -135,7 +135,10 @@ function OrganizationDetail({ organization }) {
             value={
               <ChipGroup numChips={5} totalChips={galaxy_credentials.length}>
                 {galaxy_credentials.map((credential) => (
-                  <Link to={`/credentials/${credential.id}/details`}>
+                  <Link
+                    key={credential.id}
+                    to={`/credentials/${credential.id}/details`}
+                  >
                     <CredentialChip
                       credential={credential}
                       key={credential.id}