-
Notifications
You must be signed in to change notification settings - Fork 247
feat: Add analytics for code copied, index strategy docs clicked, and updated code equivalent toggle event CLOUDP-323670 #7000
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
Conversation
| collectionName: string; | ||
| indexNameTypeMap: Record<string, string | number>; | ||
| dataTestId?: string; | ||
| onCopy?: () => void; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do want this copy tracked for the index flow section? Right now we're only doing it for the query flow section.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh wait you're right let me check
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
checked with analyst and she wanted both in and i realized i had the naming switched anyway so i fixed that. while i was there i added another event that was requested (index strategy documentation clicked)
Anemy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
| onCopy={() => { | ||
| if (onCopy) { | ||
| onCopy(); | ||
| } | ||
| }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| onCopy={() => { | |
| if (onCopy) { | |
| onCopy(); | |
| } | |
| }} | |
| onCopy={onCopy} |
Description
https://jira.mongodb.org/browse/CLOUDP-323670
News analytic event + an update to the toggle event





Checklist
Motivation and Context
Types of changes