public-offering
/
cal.pub0.org
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix embed detection (#9385)

pull/9356/head^2
Hariom Balhara 2023-06-07 14:13:12 +05:30 committed by GitHub
parent 7f1b028099
commit ff75364f22
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
packages/embeds/embed-core/embed-iframe-init.ts
View File

@ -11,10 +11,13 @@ export default function EmbedInitIframe() {
// So, we also check for the namespace in `window.name` which is set when iframe is created by embed.ts and persists for the duration of iframe's life // So, we also check for the namespace in `window.name` which is set when iframe is created by embed.ts and persists for the duration of iframe's life
// Note that, window.name isn't lost during hard navigation as well. Though, hard navigation isn't something that would happen in the app, but it's critical to be able to detect embed mode even after that(just in case) // Note that, window.name isn't lost during hard navigation as well. Though, hard navigation isn't something that would happen in the app, but it's critical to be able to detect embed mode even after that(just in case)
// We might just use window.name but if just in case something resets the `window.name`, we will still have the namespace in query param // We might just use window.name but if just in case something resets the `window.name`, we will still have the namespace in query param
// It must be null for non-embed scenario.
const embedNamespace = const embedNamespace =
typeof embedNameSpaceFromQueryParam === "string" typeof embedNameSpaceFromQueryParam === "string"
? embedNameSpaceFromQueryParam ? embedNameSpaceFromQueryParam
: window.name.replace(/cal-embed=(.*)/, "$1").trim(); : window.name.includes("cal-embed=")
? window.name.replace(/cal-embed=(.*)/, "$1").trim()
: null;
window.isEmbed = () => { window.isEmbed = () => {
// By default namespace is "". That would also work if we just check the type of variable // By default namespace is "". That would also work if we just check the type of variable