Skip to content

proactive true by default#513

Open
ruchI9897 wants to merge 5 commits intomainfrom
SCAL-307028
Open

proactive true by default#513
ruchI9897 wants to merge 5 commits intomainfrom
SCAL-307028

Conversation

@ruchI9897
Copy link
Copy Markdown
Contributor

No description provided.

@ruchI9897 ruchI9897 requested a review from a team as a code owner April 23, 2026 07:01
Copy link
Copy Markdown
Contributor

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

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 updates the initialization data in TsEmbed to provide a default value of true for embedExpiryInAuthToken when refreshAuthTokenOnNearExpiry is not provided. A review comment points out that embedExpiryInAuthToken and other properties from getHostEventsConfig are missing from the DefaultAppInitData interface in src/types.ts, which will lead to TypeScript compilation errors.

Comment thread src/embed/ts-embed.ts
Comment thread src/types.ts Outdated
Comment thread src/types.ts Outdated
Copy link
Copy Markdown
Contributor

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

Choose a reason for hiding this comment

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

LGTM

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 24, 2026

Open in StackBlitz

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

commit: bcc5a6a

sastaachar
sastaachar previously approved these changes Apr 24, 2026
Comment thread src/embed/ts-embed.ts
@ruchI9897 ruchI9897 dismissed stale reviews from sastaachar and shivam-kumar-ts via 6014877 April 24, 2026 07:30
@sonar-prod-ts
Copy link
Copy Markdown

sonar-prod-ts Bot commented Apr 24, 2026

SonarQube Quality Gate

Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

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