cal.com icon indicating copy to clipboard operation
cal.com copied to clipboard

fix(web): set canonical as `https://cal.com` in the `<HeadSeo/>`

Open p6l-richard opened this issue 2 years ago • 2 comments

⚠️ WIP This PR is currently a WIP and mostly for demo purposes

What does this PR do?

This PR handles canonicalization of the web app's pages that are deployed to both app.cal.com & cal.com (see issue).

Fixes #416

CleanShot 2023-01-25 at 11 18 28@2x

Environment: Staging(main branch) / Production

Type of change

  • [x] Bug fix (non-breaking change which fixes an issue)
  • [ ] Chore (refactoring code, technical debt, workflow improvements)
  • [ ] New feature (non-breaking change which adds functionality)
  • [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [ ] This change requires a documentation update

How should this be tested?

  • [ ] yarn test-e2e should pass
  • [ ] Inspect a couple of pages manually to re-evaluate the canonical tag

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works
  • I haven't checked if new and existing unit tests pass locally with my changes

p6l-richard avatar Jan 25 '23 14:01 p6l-richard

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
cal ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 30, 2023 at 11:53AM (UTC)
ui ❌ Failed (Inspect) Jan 30, 2023 at 11:53AM (UTC)

vercel[bot] avatar Jan 25 '23 14:01 vercel[bot]

@p6l-richard is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Jan 25 '23 14:01 vercel[bot]