SCAL-303837: Add support for enableLiveboardDataCache flag

[ajeet-lakhani-ts]

LiveboardEmbed

const tsSearch = new LiveboardEmbed("#tsEmbed", {
  frameParams: {},
  liveboardId: "<id>",
  enableLiveboardDataCache: true/false
});

AppEmbed

const tsSearch = new AppEmbed("#tsEmbed", {
     frameParams: {
     },
     enableLiveboardDataCache: true/false
   });

Embed Type	Screenshot
Enabled on LiveboardEmbed
Disabled on LiveboardEmbed
Enabled on Full App Embed
Disabled on Full App Embed

[gemini-code-assist]

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/@thoughtspot/visual-embed-sdk@492

commit: 0fed91a

[shivam-kumar-ts]

/gemini review

[gemini-code-assist]

Code Review

This pull request introduces the enablePbVizDataCaching configuration option to both AppEmbed and LiveboardEmbed, allowing users to enable visualization data caching. The implementation correctly follows the style guide by only adding the parameter to the URL when it is explicitly defined, and corresponding unit tests have been added to verify this behavior. Feedback is provided to remove redundant @type JSDoc tags in src/embed/app.ts and src/embed/liveboard.ts to align with the repository's canonical tag ordering.

[sonar-prod-ts]

SonarQube Quality Gate

0 Bugs
0 Vulnerabilities
0 Security Hotspots
2 Code Smells

No Coverage information
0.0% Duplication

[shivam-kumar-ts]

LGTM