Skip to content

Conversation

@mcasimir
Copy link
Collaborator

Description

Checklist

  • New tests and/or benchmarks are included
  • Documentation is changed or added
  • If this change updates the UI, screenshots/videos are added and a design review is requested
  • I have signed the MongoDB Contributor License Agreement (https://www.mongodb.com/legal/contributor-agreement)

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

@mcasimir mcasimir requested a review from Anemy May 27, 2025 09:33
}, [isSignInModalVisible, track]);

const handleModalClose = useCallback(() => {
track('AI Sign In Modal Dismissed', {});
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Anemy is this triggered only on dismiss or also when the user logs in?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's only triggered on dismiss, not on log in.

@mcasimir mcasimir added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label May 27, 2025
@mcasimir mcasimir merged commit 44f1ba1 into main May 30, 2025
56 checks passed
@mcasimir mcasimir deleted the track-ai-funnel branch May 30, 2025 16:01
@gribnoysup gribnoysup changed the title chore(telemetry): add tracking for AI modals and query generation funnel chore(telemetry): add tracking for AI modals and query generation funnel COMPASS-9340 Jun 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants