Skip to content

Conversation

@pallavpp
Copy link
Contributor

No description provided.

@pallavpp pallavpp force-pushed the scal-227506-override-org-id branch 3 times, most recently from 7933aae to b9a5263 Compare October 26, 2024 04:24
@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 26, 2024

Open in Stackblitz

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

commit: 8c210bd

src/types.ts Outdated
* overrideOrgId: 142536
* });
* ```
* @version SDK: 1.35 | ThoughtSpot: 10.5.0.cl
Copy link
Contributor

Choose a reason for hiding this comment

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

1.35.0

if (disableRedirectionLinksInNewTab) {
queryParams[Param.DisableRedirectionLinksInNewTab] = disableRedirectionLinksInNewTab;
}
if (overrideOrgId !== undefined) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Add UTs

@pallavpp pallavpp force-pushed the scal-227506-override-org-id branch 4 times, most recently from ee98124 to 1a7249b Compare October 28, 2024 06:15
@pallavpp pallavpp force-pushed the scal-227506-override-org-id branch from 1a7249b to 8c210bd Compare October 28, 2024 06:25
@adityamittal3107 adityamittal3107 merged commit 8794c44 into main Oct 28, 2024
4 checks passed
@pallavpp pallavpp deleted the scal-227506-override-org-id branch October 28, 2024 06:42
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