SCAL-320129 removed default home page setting

src/embed/app.spec.ts

@@ -530,7 +530,7 @@
                     homePage: HomePage.Focused,
                 },
             } as AppViewConfig,
-            `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&modularHomeExperience=false&navigationVersion=v2&homepageVersion=v4${defaultParams}${defaultParamsPost}#/home`,
+            `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&modularHomeExperience=false&homepageVersion=v4${defaultParams}${defaultParamsPost}#/home`,
         );
     });
 
@@ -1390,27 +1390,27 @@
         );
     });
 
-    test('Should add homepageVersion=v3 & navigationVersion=v3 & modularHomeExperience=true when homePage is ModularWithStylingChanges to the iframe src', async () => {
+    test('Should add homepageVersion=v3 & modularHomeExperience=false to the iframe src when homePage is ModularWithStylingChanges', async () => {
         await testUrlParams(
             {
                 ...defaultViewConfig,
                 // primaryNavbarVersion is not included under
-                // discoveryExperience, then it set navigationVersion=v2 and
-                // modularHomeExperience=false.
+                // discoveryExperience, so navigationVersion is left unset and
+                // the server's GA default applies, while modularHomeExperience is set to false.
                 discoveryExperience: {
                     homePage: HomePage.ModularWithStylingChanges,
                 },
             } as AppViewConfig,
-            `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&modularHomeExperience=false&navigationVersion=v2&homepageVersion=v3${defaultParams}${defaultParamsPost}#/home`
+            `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&modularHomeExperience=false&homepageVersion=v3${defaultParams}${defaultParamsPost}#/home`
         );
     });
 
-    test('Should add navigationVersion=v2 when primaryNavbarVersion is not added to the iframe src', async () => {
+    test('Should not add navigationVersion when primaryNavbarVersion is not added to the iframe src', async () => {
         await testUrlParams(
             {
                 ...defaultViewConfig,
             } as AppViewConfig,
-            `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&modularHomeExperience=false&navigationVersion=v2${defaultParams}${defaultParamsPost}#/home`
+            `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&modularHomeExperience=false${defaultParams}${defaultParamsPost}#/home`
         );
     });
 
@@ -1441,7 +1441,7 @@
             {
                 ...defaultViewConfig,
             } as AppViewConfig,
-            `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&modularHomeExperience=false&navigationVersion=v2${defaultParams}${defaultParamsPost}#/home`
+            `http://${thoughtSpotHost}/?embedApp=true&primaryNavHidden=true&profileAndHelpInNavBarHidden=false&modularHomeExperience=false${defaultParams}${defaultParamsPost}#/home`
         );
     });
 

src/embed/app.ts

@@ -614,7 +614,7 @@
     isPNGInScheduledEmailsEnabled?: boolean;
 
     /**
      * Enables the 'what you see is what you get' PDF export for Liveboards. Each tab is rendered on a single page
      * following the exact UI layout, instead of splitting visualizations across multiple A4 pages.
      * This feature is GA from version 26.5.0.cl. It is disabled by default in embed deployments.
      *
@@ -1181,15 +1181,9 @@
             params[Param.ModularHomeExperienceEnabled] = modularHomeExperience;
         }
 
-        // Set navigation to v2 by default to avoid problems like the app
-        // switcher (9-dot menu) not showing when v3 navigation is turned on
-        // at the cluster level.
-        // To use v3 navigation, we must manually set the discoveryExperience
-        // settings.
-        params[Param.NavigationVersion] = 'v2';
-        // Set homePageVersion to v2 by default to reset the LD flag value
-        // for the homepageVersion.
-        params[Param.HomepageVersion] = 'v2';
+        // Don't force a navigation/homepage version by default. Let the
+        // server's GA default take over (v3 on 26.8+, v2 on older clusters).
+        // Only override when the user explicitly sets discoveryExperience.
         if (discoveryExperience) {
             // primaryNavbarVersion v3 will enabled the new left navigation
             if (discoveryExperience.primaryNavbarVersion === PrimaryNavbarVersion.Sliding) {
@@ -1231,7 +1225,7 @@
             this.iFrame,
             this.viewConfig.enableScrollableContainerLazyLoading,
         );
         // this should be fired only if the lazyLoadingForFullHeight and fullHeight are true
         if(this.viewConfig.lazyLoadingForFullHeight && this.viewConfig.fullHeight){
             this.trigger(HostEvent.VisibleEmbedCoordinates, data);
         }