-
Notifications
You must be signed in to change notification settings - Fork 53
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
Track link variants property when field is added #1517
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@@ -210,6 +210,7 @@ type FieldAddedSegmentEvent = SegmentEvent< | |||
contentType: "page type" | "custom type" | "slice"; | |||
allowText?: boolean; | |||
repeat?: boolean; | |||
variants?: string[]; |
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.
❓ Interesting, so we can send an array directly to Segment / Amplitude? How does it's translated in Amplitude, as a string?
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.
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.
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.
irrc, the page builder team has some weird types for tracking events, only allowing strings
maybe it's because of the proxy we use, idk
but both segment and amplitude allow strings, numbers, arrays, booleans... which could improve a lot our analytics, imo
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!
Depends on #1516. Review that one first.
Resolves:
https://linear.app/prismic/issue/DT-2473/aapm-i-can-see-a-property-to-the-event-slice-machine-field-added-type
Description
Checklist
Preview
How to QA 1
Footnotes
Please use these labels when submitting a review:
⚠️ #issue: Strongly suggest a change.
❓ #ask: Ask a question.
💡 #idea: Suggest an idea.
🎉 #nice: Share a compliment. ↩