Skip to content

Switched coveo analytics mode from default EP to legacy #170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 12, 2025

Conversation

lamATnginx
Copy link
Collaborator

Proposed changes

Closes https://github.com/nginxinc/docs-platform/issues/387

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING document
  • If applicable, I have added tests that prove my fix is effective or that my feature works
  • If applicable, I have checked that any relevant tests pass after adding my changes
  • I have updated any relevant documentation (README.md and CHANGELOG.md)

@lamATnginx lamATnginx self-assigned this Mar 12, 2025
@lamATnginx lamATnginx requested review from a team as code owners March 12, 2025 13:29
@lamATnginx lamATnginx changed the title fix: Switched analytics mode from default EP to legacy Switched analytics mode from default EP to legacy Mar 12, 2025
@lamATnginx lamATnginx force-pushed the fix/coveo-analytics-mode branch from 1fee6fe to 89c0871 Compare March 12, 2025 13:37
@lamATnginx
Copy link
Collaborator Author

lamATnginx commented Mar 12, 2025

Deployed to dev. Check-out the Network tab in https://docs-dev.nginx.com/search.html#q=proxy versus https://docs.nginx.com/search.html#q=proxy to see search events are being properly registered for Coveo. Example:

On an operation to just click a link:

Before:
Screenshot 2025-03-12 at 6 51 14 AM

After:
Screenshot 2025-03-12 at 6 51 06 AM

@lamATnginx lamATnginx changed the title Switched analytics mode from default EP to legacy Switched coveo analytics mode from default EP to legacy Mar 12, 2025
@lamATnginx lamATnginx merged commit f9a15f3 into main Mar 12, 2025
5 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.

2 participants