Skip to content

Conversation

@wofai-ewa
Copy link
Contributor

Clarity was previously hardcoded and firing regardless of user consent. This update wraps the Clarity script in a Cookiebot check, ensuring it only runs when “Marketing” cookies are accepted. Aligns Clarity behavior with GTM-based VWO consent setup.

Please include a summary of the change and what is does. Please also include relevant motivation and context.

Review process:

  • No need to request review from any specific docs team members; we will see your PR and one of us will review.
  • If you need technical review from engineering, please request review from the relevant person.
  • Docs team might commit styling nit fixes to save you the trouble :)
  • You can merge after approval from one docs team member (as well as any necessary technical review).

Clarity was previously hardcoded and firing regardless of user consent.
This update wraps the Clarity script in a Cookiebot check, ensuring it only runs when “Marketing” cookies are accepted.
Aligns Clarity behavior with GTM-based VWO consent setup.
@netlify
Copy link

netlify bot commented Nov 26, 2025

Deploy Preview for viam-docs ready!

Name Link
🔨 Latest commit 8f9e201
🔍 Latest deploy log https://app.netlify.com/projects/viam-docs/deploys/6927854b49b4140008136f8a
😎 Deploy Preview https://deploy-preview-4767--viam-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 58 (🟢 up 16 from production)
Accessibility: 100 (no change from production)
Best Practices: 100 (no change from production)
SEO: 92 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@viambot
Copy link
Member

viambot commented Nov 26, 2025

👋 Thanks for contributing! A reviewer will look at it on the next working day!

@npentrel npentrel merged commit 6c62e7d into viamrobotics:main Nov 27, 2025
11 of 12 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.

4 participants