@thoughtspot/visual-embed-sdk 1.27.0-alpha.0 → 1.27.0-logger

package/lib/src/utils/logger.spec.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"logger.spec.js","sourceRoot":"","sources":["../../../src/utils/logger.spec.ts"],"names":[],"mappings":"AAAA,IAAI,MAAW,CAAC;AAChB,IAAI,QAAa,CAAC;AAClB,IAAI,yBAA8B,CAAC;AAEnC,MAAM,eAAe,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC;AACrD,MAAM,cAAc,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;AACnD,MAAM,cAAc,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;AACnD,MAAM,eAAe,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC;AACrD,MAAM,eAAe,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,EAAE,OAAO,CAAC,CAAC;AAErD,QAAQ,CAAC,QAAQ,EAAE,GAAG,EAAE;IACpB,SAAS,CAAC,KAAK,IAAI,EAAE;QACjB,MAAM,CAAC,GAAG,MAAM,MAAM,CAAC,UAAU,CAAC,CAAC;QACnC,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;QAClB,QAAQ,GAAG,CAAC,CAAC,QAAQ,CAAC;QACtB,yBAAyB,GAAG,CAAC,CAAC,yBAAyB,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,UAAU,CAAC,GAAG,EAAE;QACZ,gDAAgD;QAChD,yBAAyB,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;QAC1C,MAAM,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;QAEnC,eAAe,CAAC,WAAW,EAAE,CAAC;QAC9B,cAAc,CAAC,WAAW,EAAE,CAAC;QAC7B,cAAc,CAAC,WAAW,EAAE,CAAC;QAC7B,eAAe,CAAC,WAAW,EAAE,CAAC;QAC9B,eAAe,CAAC,WAAW,EAAE,CAAC;IAClC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC1C,MAAM,CAAC,WAAW,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QAClC,MAAM,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QAEjD,MAAM,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;QACnC,MAAM,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4CAA4C,EAAE,GAAG,EAAE;QAClD,MAAM,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;QACnC,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;QAC9B,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAAC,eAAe,CAAC,CAAC;QAE9D,MAAM,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;QAC/B,MAAM,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAE/C,MAAM,CAAC,WAAW,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QAClC,MAAM,CAAC,KAAK,CAAC,iBAAiB,CAAC,CAAC;QAChC,MAAM,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;QAC/B,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAAC,iBAAiB,CAAC,CAAC;QAChE,MAAM,CAAC,cAAc,CAAC,CAAC,oBAAoB,CAAC,iBAAiB,CAAC,CAAC;IACnE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,GAAG,EAAE;QAC9D,yBAAyB,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC;QAEzC,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;QAC9B,MAAM,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;QAC5B,MAAM,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;QAC5B,MAAM,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;QAC9B,MAAM,CAAC,eAAe,CAAC,CAAC,oBAAoB,CAAC,eAAe,CAAC,CAAC;QAC9D,MAAM,CAAC,cAAc,CAAC,CAAC,oBAAoB,CAAC,cAAc,CAAC,CAAC;QAC5D,MAAM,CAAC,cAAc,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAC9C,MAAM,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QAC/C,MAAM,CAAC,eAAe,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACnD,CAAC,CAAC,CAAC;AACP,CAAC,CAAC,CAAC"}

package/lib/src/visual-embed-sdk.d.ts

@@ -18,12 +18,12 @@ declare module '@thoughtspot/visual-embed-sdk' {
     import { SearchEmbed, SearchViewConfig } from '@thoughtspot/visual-embed-sdk/embed/search';
     import { SearchBarEmbed, SearchBarViewConfig } from '@thoughtspot/visual-embed-sdk/embed/search-bar';
     import { AuthFailureType, AuthStatus, AuthEvent, AuthEventEmitter, getSessionInfo } from '@thoughtspot/visual-embed-sdk/auth';
-    import { AuthType, RuntimeFilter, RuntimeFilterOp, EmbedEvent, HostEvent, DataSourceVisualMode, Action, EmbedConfig, PrefetchFeatures, FrameParams, DOMSelector, HomeLeftNavItem, HomepageModule, MessageOptions, MessageCallback, MessagePayload, CustomisationsInterface, CustomStyles, customCssInterface, ContextMenuTriggerOptions, RuntimeParameter } from '@thoughtspot/visual-embed-sdk/types';
+    import { AuthType, RuntimeFilter, RuntimeFilterOp, EmbedEvent, HostEvent, DataSourceVisualMode, Action, EmbedConfig, PrefetchFeatures, FrameParams, DOMSelector, HomeLeftNavItem, HomepageModule, MessageOptions, MessageCallback, MessagePayload, CustomisationsInterface, CustomStyles, customCssInterface, ContextMenuTriggerOptions, RuntimeParameter, LogLevel } from '@thoughtspot/visual-embed-sdk/types';
     import { CustomCssVariables } from '@thoughtspot/visual-embed-sdk/css-variables';
     import { SageEmbed, SageViewConfig } from '@thoughtspot/visual-embed-sdk/embed/sage';
     import { AnswerService } from '@thoughtspot/visual-embed-sdk/utils/graphql/answerService/answerService';
     import { getEmbedConfig } from '@thoughtspot/visual-embed-sdk/embed/embedConfig';
-    export { init, logout, prefetch, executeTML, exportTML, executeTMLInput, exportTMLInput, getEmbedConfig as getInitConfig, getSessionInfo, SearchEmbed, SearchBarEmbed, PinboardEmbed, LiveboardEmbed, SageEmbed, AppEmbed, AuthFailureType, AuthStatus, AuthEvent, AuthEventEmitter, AnswerService, Page, AuthType, RuntimeFilter, RuntimeFilterOp, EmbedEvent, HostEvent, DataSourceVisualMode, Action, ContextMenuTriggerOptions, EmbedConfig, SearchViewConfig, SearchBarViewConfig, LiveboardViewConfig, SageViewConfig, AppViewConfig, PrefetchFeatures, FrameParams, HomeLeftNavItem, HomepageModule, DOMSelector, MessageOptions, MessageCallback, MessagePayload, CustomisationsInterface, CustomStyles, customCssInterface, CustomCssVariables, RuntimeParameter, };
+    export { init, logout, prefetch, executeTML, exportTML, executeTMLInput, exportTMLInput, getEmbedConfig as getInitConfig, getSessionInfo, SearchEmbed, SearchBarEmbed, PinboardEmbed, LiveboardEmbed, SageEmbed, AppEmbed, AuthFailureType, AuthStatus, AuthEvent, AuthEventEmitter, AnswerService, Page, AuthType, RuntimeFilter, RuntimeFilterOp, EmbedEvent, HostEvent, DataSourceVisualMode, Action, ContextMenuTriggerOptions, EmbedConfig, SearchViewConfig, SearchBarViewConfig, LiveboardViewConfig, SageViewConfig, AppViewConfig, PrefetchFeatures, FrameParams, HomeLeftNavItem, HomepageModule, DOMSelector, MessageOptions, MessageCallback, MessagePayload, CustomisationsInterface, CustomStyles, customCssInterface, CustomCssVariables, RuntimeParameter, LogLevel, };
 }
 
 declare module '@thoughtspot/visual-embed-sdk/embed/app' {
@@ -227,12 +227,6 @@ declare module '@thoughtspot/visual-embed-sdk/embed/app' {
                 * embedded Liveboard or visualization.
                 */
             protected getEmbedParams(): string;
-            /**
-                * Constructs the URL of the ThoughtSpot app page to be rendered.
-                *
-                * @param pageId The ID of the page to be embedded.
-                */
-            getIFrameSrc(): string;
             /**
                 * Set the iframe height as per the computed height received
                 * from the ThoughtSpot app.
@@ -729,14 +723,6 @@ declare module '@thoughtspot/visual-embed-sdk/embed/search' {
             protected embedComponentType: string;
             constructor(domSelector: DOMSelector, viewConfig: SearchViewConfig);
             protected getEmbedParams(): string;
-            /**
-                * Construct the URL of the embedded ThoughtSpot search to be
-                * loaded in the iframe
-                *
-                * @param answerId The GUID of a saved answer
-                * @param dataSources A list of data source GUIDs
-                */
-            getIFrameSrc(): string;
             /**
                 * Render the embedded ThoughtSpot search
                 */
@@ -1512,6 +1498,20 @@ declare module '@thoughtspot/visual-embed-sdk/types' {
                 * @version SDK: 1.26.2 | ThoughtSpot: *
                 */
             suppressErrorAlerts?: boolean;
+            /**
+                * Log level for the SDK.
+                * @default LogLevel.ERROR
+                *
+                * @example
+                * ```js
+                * init({
+                *   ...embedConfig,
+                *   logLevel: LogLevel.SILENT
+                * })
+                * ```
+                * @version SDK: 1.26.3 | ThoughtSpot: 9.10.0.cl
+                */
+            logLevel?: LogLevel;
     }
     export interface LayoutConfig {
     }
@@ -3755,7 +3755,9 @@ declare module '@thoughtspot/visual-embed-sdk/types' {
             IsProductTour = "isProductTour",
             HideSearchBarTitle = "hideSearchBarTitle",
             HideSageAnswerHeader = "hideSageAnswerHeader",
-            HideSearchBar = "hideSearchBar"
+            HideSearchBar = "hideSearchBar",
+            ClientLogLevel = "clientLogLevel",
+            OverrideNativeConsole = "overrideConsoleLogs"
     }
     /**
         * ThoughtSpot application pages include actions and menu commands
@@ -4670,6 +4672,14 @@ declare module '@thoughtspot/visual-embed-sdk/types' {
             session: SessionInterface;
             vizId?: string;
     }
+    export enum LogLevel {
+            SILENT = "SILENT",
+            ERROR = "ERROR",
+            WARN = "WARN",
+            INFO = "INFO",
+            DEBUG = "DEBUG",
+            TRACE = "TRACE"
+    }
 }
 
 declare module '@thoughtspot/visual-embed-sdk/css-variables' {
@@ -5166,13 +5176,6 @@ declare module '@thoughtspot/visual-embed-sdk/embed/sage' {
                 * @returns {string} query string
                 */
             protected getEmbedParams(): string;
-            /**
-                * Construct the URL of the embedded ThoughtSpot sage to be
-                * loaded in the iframe
-                *
-                * @returns {string} iframe url
-                */
-            getIFrameSrc(): string;
             /**
                 * Render the embedded ThoughtSpot Sage
                 *
@@ -5270,12 +5273,13 @@ declare module '@thoughtspot/visual-embed-sdk/embed/embedConfig' {
     export const getEmbedConfig: () => EmbedConfig;
     /**
         * Sets the configuration embed was initialized with.
+        * And returns the new configuration.
         *
         * @param newConfig The configuration to set.
         * @version SDK: 1.27.0 | ThoughtSpot: *
         * @group Global methods
         */
-    export const setEmbedConfig: (newConfig: EmbedConfig) => void;
+    export const setEmbedConfig: (newConfig: EmbedConfig) => EmbedConfig;
 }
 
 declare module '@thoughtspot/visual-embed-sdk/embed/ts-embed' {
@@ -5475,7 +5479,6 @@ declare module '@thoughtspot/visual-embed-sdk/embed/ts-embed' {
                 * @param args
                 */
             render(): TsEmbed;
-            getIframeSrc(): string;
             protected handleRenderForPrerender(): void;
             /**
                 * Creates the preRender shell

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@thoughtspot/visual-embed-sdk",
-    "version": "1.27.0-alpha.0",
+    "version": "1.27.0-logger",
     "description": "ThoughtSpot Embed SDK",
     "module": "lib/src/index.js",
     "main": "dist/tsembed.js",

package/src/auth.ts

@@ -13,6 +13,7 @@ import {
     EndPoints,
 } from './utils/authService';
 import { getAuthenticationToken, resetCachedAuthToken } from './authToken';
+import { logger } from './utils/logger';
 
 // eslint-disable-next-line import/no-mutable-exports
 export let loggedInStatus = false;
@@ -154,7 +155,7 @@ export function setAuthEE(eventEmitter: EventEmitter<AuthStatus | AuthEvent>): v
  */
 export function notifyAuthSDKSuccess(): void {
     if (!authEE) {
-        console.error('SDK not initialized');
+        logger.error('SDK not initialized');
         return;
     }
     authEE.emit(AuthStatus.SDK_SUCCESS);
@@ -165,7 +166,7 @@ export function notifyAuthSDKSuccess(): void {
  */
 export function notifyAuthSuccess(): void {
     if (!authEE) {
-        console.error('SDK not initialized');
+        logger.error('SDK not initialized');
         return;
     }
     authEE.emit(AuthStatus.SUCCESS, sessionInfo);
@@ -177,7 +178,7 @@ export function notifyAuthSuccess(): void {
  */
 export function notifyAuthFailure(failureType: AuthFailureType): void {
     if (!authEE) {
-        console.error('SDK not initialized');
+        logger.error('SDK not initialized');
         return;
     }
     authEE.emit(AuthStatus.FAILURE, failureType);
@@ -188,7 +189,7 @@ export function notifyAuthFailure(failureType: AuthFailureType): void {
  */
 export function notifyLogout(): void {
     if (!authEE) {
-        console.error('SDK not initialized');
+        logger.error('SDK not initialized');
         return;
     }
     authEE.emit(AuthStatus.LOGOUT);

package/src/embed/app.spec.ts

@@ -18,6 +18,7 @@ import {
 import { version } from '../../package.json';
 import * as config from '../config';
 import { TsEmbed, V1Embed } from './ts-embed';
+import { logger } from '../utils/logger';
 
 const defaultViewConfig = {
     frameParams: {
@@ -463,15 +464,15 @@ describe('App embed tests', () => {
                 },
             });
             await appEmbed.render();
-            spyOn(console, 'warn');
+            spyOn(logger, 'warn');
             appEmbed.navigateToPage(-1);
-            expect(console.warn).toHaveBeenCalledWith(
+            expect(logger.warn).toHaveBeenCalledWith(
                 'Path can only by a string when triggered without noReload',
             );
         });
 
         test('navigateToPage function use before render', async () => {
-            spyOn(console, 'log');
+            spyOn(logger, 'log');
             const appEmbed = new AppEmbed(getRootEl(), {
                 frameParams: {
                     width: '100%',
@@ -480,7 +481,7 @@ describe('App embed tests', () => {
             });
             appEmbed.navigateToPage(path);
             await appEmbed.render();
-            expect(console.log).toHaveBeenCalledWith(
+            expect(logger.log).toHaveBeenCalledWith(
                 'Please call render before invoking this method',
             );
         });

package/src/embed/app.ts

@@ -9,6 +9,7 @@
  * @author Ayon Ghosh <ayon.ghosh@thoughtspot.com>
  */
 
+import { logger } from '../utils/logger';
 import { getQueryParamString } from '../utils';
 import {
     Param, DOMSelector, HostEvent, ViewConfig, EmbedEvent, MessagePayload,
@@ -282,7 +283,7 @@ export class AppEmbed extends V1Embed {
      *
      * @param pageId The ID of the page to be embedded.
      */
-    public getIFrameSrc(): string {
+    private getIFrameSrc() {
         const {
             pageId, path, modularHomeExperience,
         } = this.viewConfig;
@@ -376,14 +377,14 @@ export class AppEmbed extends V1Embed {
      */
     public navigateToPage(path: string | number, noReload = false): void {
         if (!this.iFrame) {
-            console.log('Please call render before invoking this method');
+            logger.log('Please call render before invoking this method');
             return;
         }
         if (noReload) {
             this.trigger(HostEvent.Navigate, path);
         } else {
             if (typeof path !== 'string') {
-                console.warn('Path can only by a string when triggered without noReload');
+                logger.warn('Path can only by a string when triggered without noReload');
                 return;
             }
             const iframeSrc = this.iFrame.src;

package/src/embed/base.spec.ts

@@ -17,6 +17,7 @@ import {
     getIFrameSrc,
 } from '../test/test-utils';
 import * as tokenizedFetchInstance from '../tokenizedFetch';
+import { logger } from '../utils/logger';
 
 const thoughtSpotHost = 'tshost';
 let authEE: EventEmitter;
@@ -412,11 +413,11 @@ describe('Base TS Embed', () => {
 describe('Base without init', () => {
     test('notify should error when called without init', () => {
         base.reset();
-        jest.spyOn(global.console, 'error').mockImplementation(() => undefined);
+        jest.spyOn(logger, 'error').mockImplementation(() => undefined);
         base.notifyAuthSuccess();
         base.notifyAuthFailure(auth.AuthFailureType.SDK);
         base.notifyLogout();
         base.notifyAuthSDKSuccess();
-        expect(global.console.error).toHaveBeenCalledTimes(4);
+        expect(logger.error).toHaveBeenCalledTimes(4);
     });
 });

package/src/embed/base.ts

@@ -10,10 +10,13 @@
  */
 import EventEmitter from 'eventemitter3';
 import uniq from 'lodash/uniq';
+import { logger, setGlobalLogLevelOverride } from '../utils/logger';
 import { tokenizedFetch } from '../tokenizedFetch';
 import { EndPoints } from '../utils/authService/authService';
 import { getThoughtSpotHost } from '../config';
-import { AuthType, EmbedConfig, PrefetchFeatures } from '../types';
+import {
+    AuthType, EmbedConfig, LogLevel, PrefetchFeatures,
+} from '../types';
 import {
     authenticate,
     logout as _logout,
@@ -34,6 +37,7 @@ const CONFIG_DEFAULTS: Partial<EmbedConfig> = {
     loginFailedMessage: 'Not logged in',
     authTriggerText: 'Authorize',
     authType: AuthType.None,
+    logLevel: LogLevel.ERROR,
 };
 
 export interface executeTMLInput {
@@ -100,7 +104,7 @@ const hostUrlToFeatureUrl = {
 export const prefetch = (url?: string, prefetchFeatures?: PrefetchFeatures[]): void => {
     if (url === '') {
         // eslint-disable-next-line no-console
-        console.warn('The prefetch method does not have a valid URL');
+        logger.warn('The prefetch method does not have a valid URL');
     } else {
         const features = prefetchFeatures || [PrefetchFeatures.FullApp];
         let hostUrl = url || getEmbedConfig().thoughtSpotHost;
@@ -174,13 +178,16 @@ function backwardCompat(embedConfig: EmbedConfig): EmbedConfig {
  */
 export const init = (embedConfig: EmbedConfig): AuthEventEmitter => {
     sanity(embedConfig);
-    setEmbedConfig(
+    embedConfig = setEmbedConfig(
         backwardCompat({
             ...CONFIG_DEFAULTS,
             ...embedConfig,
             thoughtSpotHost: getThoughtSpotHost(embedConfig),
         }),
     );
+
+    setGlobalLogLevelOverride(embedConfig.logLevel);
+
     const authEE = new EventEmitter<AuthStatus | AuthEvent>();
     setAuthEE(authEE);
     handleAuth();

package/src/embed/embed.spec.ts

@@ -1,3 +1,4 @@
+import { logger } from '../utils/logger';
 import {
     init, AuthType, SearchEmbed, EmbedEvent,
 } from '../index';
@@ -51,7 +52,7 @@ describe('test view config', () => {
     });
 
     test('trying to register event handler after render should throw error', async () => {
-        spyOn(console, 'error');
+        spyOn(logger, 'error');
         const onErrorSpy = jest.fn();
         const searchEmbed = new SearchEmbed(getRootEl(), defaultViewConfig);
         searchEmbed

package/src/embed/embedConfig.ts

@@ -13,6 +13,7 @@ export const getEmbedConfig = (): EmbedConfig => config;
 
 /**
  * Sets the configuration embed was initialized with.
+ * And returns the new configuration.
  *
  * @param newConfig The configuration to set.
  * @version SDK: 1.27.0 | ThoughtSpot: *
@@ -20,4 +21,5 @@ export const getEmbedConfig = (): EmbedConfig => config;
  */
 export const setEmbedConfig = (newConfig: EmbedConfig) => {
     config = newConfig;
+    return newConfig;
 };

package/src/embed/events.spec.ts

@@ -1,3 +1,4 @@
+import { logger } from '../utils/logger';
 import {
     init,
     AuthType,
@@ -155,7 +156,7 @@ describe('test communication between host app and ThoughtSpot', () => {
             pinboardId: 'eca215d4-0d2c-4a55-90e3-d81ef6848ae0',
         } as LiveboardViewConfig);
         liveboardEmbed.render();
-        const spy1 = jest.spyOn(global.console, 'log');
+        const spy1 = jest.spyOn(logger, 'log');
 
         await executeAfterWait(() => {
             const iframe = getIFrameEl();
@@ -173,7 +174,7 @@ describe('test communication between host app and ThoughtSpot', () => {
             pinboardId: 'eca215d4-0d2c-4a55-90e3-d81ef6848ae0',
         } as LiveboardViewConfig);
         pinboardEmbed.render();
-        const spy1 = jest.spyOn(global.console, 'log');
+        const spy1 = jest.spyOn(logger, 'log');
 
         await executeAfterWait(() => {
             const iframe = getIFrameEl();

package/src/embed/liveboard.ts

@@ -28,7 +28,7 @@ import { V1Embed } from './ts-embed';
  *
  * @group Embed components
  */
-export interface LiveboardViewConfig extends Omit<ViewConfig, 'hiddenHomepageModules' | 'hiddenHomeLeftNavItems' | 'reorderedHomepageModules'> {
+export interface LiveboardViewConfig extends Omit<ViewConfig, 'hiddenHomepageModules' | 'hiddenHomeLeftNavItems'| 'reorderedHomepageModules'> {
     /**
      * If set to true, the embedded object container dynamically resizes
      * according to the height of the Liveboard.
@@ -263,7 +263,7 @@ export class LiveboardEmbed extends V1Embed {
      * Construct the URL of the embedded ThoughtSpot Liveboard or visualization
      * to be loaded within the iFrame.
      */
-    private getIFrameSrc(): string {
+    private getIFrameSrc() {
         const { vizId, activeTabId } = this.viewConfig;
         const liveboardId = this.viewConfig.liveboardId ?? this.viewConfig.pinboardId;
 
@@ -312,7 +312,7 @@ export class LiveboardEmbed extends V1Embed {
         if (isUndefined(embedObj)) return;
 
         const showDifferentLib = this.viewConfig.liveboardId
-            && embedObj.viewConfig.liveboardId !== this.viewConfig.liveboardId;
+        && embedObj.viewConfig.liveboardId !== this.viewConfig.liveboardId;
 
         if (showDifferentLib) {
             const libId = this.viewConfig.liveboardId;
@@ -379,4 +379,4 @@ export class LiveboardEmbed extends V1Embed {
 /**
  * @hidden
  */
-export class PinboardEmbed extends LiveboardEmbed { }
+export class PinboardEmbed extends LiveboardEmbed {}

package/src/embed/sage.ts

@@ -212,7 +212,7 @@ export class SageEmbed extends V1Embed {
      *
      * @returns {string} iframe url
      */
-    public getIFrameSrc(): string {
+    private getIFrameSrc() {
         const path = 'eureka';
         const postHashObj = {};
         const tsPostHashParams = this.getThoughtSpotPostUrlParams();

package/src/embed/search.ts

@@ -272,8 +272,7 @@ export class SearchEmbed extends TsEmbed {
      * @param answerId The GUID of a saved answer
      * @param dataSources A list of data source GUIDs
      */
-    public getIFrameSrc(): string {
-        const { answerId } = this.viewConfig;
+    private getIFrameSrc(answerId: string) {
         const answerPath = answerId ? `saved-answer/${answerId}` : 'answer';
         const tsPostHashParams = this.getThoughtSpotPostUrlParams();
 
@@ -287,14 +286,14 @@ export class SearchEmbed extends TsEmbed {
         super.render();
         const { answerId } = this.viewConfig;
 
-        const src = this.getIFrameSrc();
+        const src = this.getIFrameSrc(answerId);
         this.renderIFrame(src);
         getAuthPromise().then(() => {
             if (
                 checkReleaseVersionInBeta(
                     getReleaseVersion(),
                     getEmbedConfig().suppressSearchEmbedBetaWarning
-                    || getEmbedConfig().suppressErrorAlerts,
+                        || getEmbedConfig().suppressErrorAlerts,
                 )
             ) {
                 alert(ERROR_MESSAGE.SEARCHEMBED_BETA_WRANING_MESSAGE);