Skip to content

SCAL-301123: Support Hiding the "Customize Homepage" Action in Embeds - #681

Merged
tushardeepakts merged 2 commits into
mainfrom
SCAL-301123_homepage_button
Sep 23, 2026
Merged

tushardeepakts merged 2 commits into
mainfrom
SCAL-301123_homepage_button

Conversation

@tushardeepakts

@tushardeepakts tushardeepakts commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds CustomizeHomepage = 'customizeHomepage' to the Action enum so the "Customize homepage" button can be hidden via the standard hiddenActions mechanism without requiring a one-off flag.

Changes

  • Add Action.CustomizeHomepage to the Action enum (no extra AppEmbed logic required as hiddenActions is already generic).
  • Add unit test asserting hiddenActions: [Action.CustomizeHomepage] emits hideAction=[...,"customizeHomepage"] in the iframe src.
Screen.Recording.2026-09-23.at.12.19.12.PM.mov

@tushardeepakts tushardeepakts self-assigned this Sep 22, 2026
@tushardeepakts
tushardeepakts requested a review from a team as a code owner September 22, 2026 10:17

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces the hideCustomizeHomepageButton configuration option to AppEmbed, enabling users to hide the 'Customize homepage' button on the embedded home page. The changes include updating the AppViewConfig interface, passing the parameter to the iframe URL when defined, updating the Param enum, and adding a unit test. The review feedback correctly identifies a style guide violation in the JSDoc code example for hideCustomizeHomepageButton, where invalid syntax (...) was used inside an object literal.

Comment thread src/embed/app.ts Outdated
@pkg-pr-new

pkg-pr-new Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

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

commit: 30441f9

@shivam-kumar-ts shivam-kumar-ts left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tushardeepakts tushardeepakts changed the title SCAL-301123: Add hideCustomizeHomepageButton to AppEmbed SCAL-301123: Support Hiding the "Customize Homepage" Action in Embeds Sep 23, 2026
@tushardeepakts
tushardeepakts merged commit 0ec54b4 into main Sep 23, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants