Use `link_url` in Lunr search-result click events, only in production

vitessio / website

vitess.io website and other project management related content

https://vitess.io

Apache License 2.0

48 stars 187 forks source link

Closed chalin closed 10 months ago

chalin commented 10 months ago

Contributes to #1570
For search click events:
- Switches to using event param link_url rather than value to pass the searchResultURL
- Sets outbound to false
Ensures that GA4 search and search-result-click events are emitted only from production built sites

/cc @nate-double-u @harshit-gangal

netlify[bot] commented 10 months ago

Name	Link
Latest commit	15fed76fec8a561f647a06690eeab0c2d6a0aa8a
Latest deploy log	https://app.netlify.com/sites/vitess/deploys/64ee74c82683480008d2ddf6
Deploy Preview	https://deploy-preview-1571--vitess.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

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

netlify[bot] commented 10 months ago

Name	Link
Latest commit	32b5170bbc92aac6c065ec93565e081dfb11d651
Latest deploy log	https://app.netlify.com/sites/vitess/deploys/64ee74ddb0709a0008fe1823
Deploy Preview	https://deploy-preview-1571--vitess.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

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