Fixed a couple places where we were not URI encoding groupId and artifactId in nav links; (#5470)

Author: EricWittmann

ui/ui-app/src/app/pages/artifact/ArtifactPage.tsx

@@ -97,10 +97,13 @@ export const ArtifactPage: FunctionComponent<PageProperties> = () => {
     };
 
     const handleTabClick = (_event: any, tabIndex: any): void => {
+        const gid: string = encodeURIComponent(groupId as string);
+        const aid: string = encodeURIComponent(artifactId as string);
+
         if (tabIndex === "overview") {
-            appNavigation.navigateTo(`/explore/${groupId}/${artifactId}`);
+            appNavigation.navigateTo(`/explore/${gid}/${aid}`);
         } else {
-            appNavigation.navigateTo(`/explore/${groupId}/${artifactId}/${tabIndex}`);
+            appNavigation.navigateTo(`/explore/${gid}/${aid}/${tabIndex}`);
         }
     };
 

ui/ui-app/src/app/pages/group/GroupPage.tsx

@@ -83,10 +83,12 @@ export const GroupPage: FunctionComponent<PageProperties> = () => {
     };
 
     const handleTabClick = (_event: any, tabIndex: any): void => {
+        const gid: string = encodeURIComponent(groupId as string);
+
         if (tabIndex === "overview") {
-            appNavigation.navigateTo(`/explore/${groupId}`);
+            appNavigation.navigateTo(`/explore/${gid}`);
         } else {
-            appNavigation.navigateTo(`/explore/${groupId}/${tabIndex}`);
+            appNavigation.navigateTo(`/explore/${gid}/${tabIndex}`);
         }
     };