handle window undefined during pre-render

pull/6802/head
Richard Poelderl 2023-01-31 10:34:29 -03:00
parent 3208522c70
commit 346daafcdb
1 changed files with 12 additions and 8 deletions

View File

@ -73,14 +73,18 @@ const buildSeoMeta = (pageProps: {
}; };
export const HeadSeo = (props: HeadSeoProps): JSX.Element => { export const HeadSeo = (props: HeadSeoProps): JSX.Element => {
// build the canonical url to ensure it's always cal.com (not app.cal.com) // The below code sets the defaultUrl for our canonical tags
const router = useRouter();
// compose the url with only the router's path (e.g. /apps/zapier) such that on app.cal.com the canonical is still cal.com // Get the current URL from the window object
const calcomUrl = (`https://cal.com` + (router.asPath === "/" ? "" : router.asPath)).split("?")[0]; // cut off search params const url = getBrowserInfo()?.url;
const url = getBrowserInfo()?.url ?? ""; // Check if the URL is from cal.com
// avoid setting cal.com canonicals on self-hosted apps. Note: isCalcom or IS_SELF_HOSTED from @calcom/lib do not handle https:cal.com const isCalcom = url && new URL(url).hostname.endsWith("cal.com");
const isCalcom = new URL(url).hostname.endsWith("cal.com"); // Get the router's path
const defaultUrl = isCalcom ? calcomUrl : url; const path = useRouter().asPath;
// Build the canonical URL using the router's path, without query parameters. Note: on homepage it omits the trailing slash
const calcomCanonical = `https://cal.com${path === "/" ? "" : path}`.split("?")[0];
// Set the default URL to either the current URL (if self-hosted) or https://cal.com canonical URL
const defaultUrl = isCalcom ? calcomCanonical : url;
const { title, description, siteName, canonical = defaultUrl, nextSeoProps = {}, app, meeting } = props; const { title, description, siteName, canonical = defaultUrl, nextSeoProps = {}, app, meeting } = props;