action-destinations icon indicating copy to clipboard operation
action-destinations copied to clipboard

hide event context field for mixpanel

Open tcgilbert opened this issue 1 year ago • 2 comments

This comes at the request of Mixpanel:

Hi all!

This is Donna from Mixpanel's Support Team.

I wanted to reach out regarding Mixpanel Actions Destination Mappings: https://segment.com/docs/connections/destinations/catalog/actions-mixpanel/

We are seeing that customers are confused by the "Event Context" that is shown in Track Calls.

Customers see that mapping, and they are confused as to why Mixpanel does not attach all of "event context" data to their Mixpanel Events.

With the Destination, we are only taking certain data from the Event Context (as Context is not a concept in Mixpanel) so I wanted to ask if we could hide this mapping in the UI for Mixpanel Actions? image.png

We've also previously brought this up in the shared Slack channel we have (#segment_mixpanel_integration) if you'd like to continue the conversation via Slack.

I can't think of a use case for exposing this, other than if a user wants to NOT pass any context details. So open to discussion on this.

tcgilbert avatar Apr 25 '24 19:04 tcgilbert

Hi @tcgilbert , How is the data from the context field actually being processed in the perform() function? Are parts of it used and others discarded?

joe-ayoub-segment avatar Apr 26 '24 08:04 joe-ayoub-segment

How is the data from the context field actually being processed in the perform() function? Are parts of it used and others discarded?

@joe-ayoub-segment so this property is imported into trackEvent and trackPurchase, but ultimately it gets parsed in the same function getEventProperties. Within this function, the only property that is being pulled from context is payload.context?.integration if it exists.

That being the case, I'd say the need for this field is a bit overkill in the first place.

tcgilbert avatar Apr 30 '24 12:04 tcgilbert

hi @tcgilbert PR deployed. Please check it all looks good!

joe-ayoub-segment avatar May 28 '24 12:05 joe-ayoub-segment

looks good, thanks!

tcgilbert avatar May 28 '24 13:05 tcgilbert