Skip to content

added new embed events#491

Merged
ruchI9897 merged 3 commits intomainfrom
embed-event-update
Apr 1, 2026
Merged

added new embed events#491
ruchI9897 merged 3 commits intomainfrom
embed-event-update

Conversation

@ruchI9897
Copy link
Copy Markdown
Contributor

No description provided.

@ruchI9897 ruchI9897 requested a review from a team as a code owner March 31, 2026 09:58
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 31, 2026

Open in StackBlitz

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

commit: 20cbeb6

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 adds the subscribedEvent helper method to the TsEmbed class and introduces EmbedPageContextChanged and Subscribed to the EmbedEvent enum. The review feedback highlights several JSDoc style guide violations, specifically regarding the canonical ordering of tags and the requirement for complete sentences to end with periods. There is also a correction for a code example that incorrectly uses an instance method as a global function, along with a suggestion to add a trailing comma to the enum definition for consistency.

@sonar-prod-ts
Copy link
Copy Markdown

sonar-prod-ts bot commented Mar 31, 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

@ruchI9897 ruchI9897 merged commit ad491a8 into main Apr 1, 2026
10 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.

2 participants