Skip to content
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

[Mixpane] Add info that events might failed due to message Id #6994

Merged
merged 2 commits into from
Sep 19, 2024

Conversation

tanjinhong72
Copy link
Contributor

Proposed changes

Customer had encountered error sending events to Mixpanel when they generated their own message Id.

Add the following info

"Failed events due to message Id

By default, message Id of an event is map to insert ID in Mixpanel. If you are generating your own message ID, check that the message Id comply to the format of Insert ID as mentioned in the document in Mixpanel. Otherwise, this will result in 400 Bad Request error from Mixpanel."

Merge timing

Related issues (optional)

https://segment.zendesk.com/agent/tickets/549513

Add the following information
------------------------------

### Failed events due to message Id

By default, message Id of an event is map to insert ID in Mixpanel. If you are generating your own message ID, check that the message Id comply to the format of Insert ID as mentioned in the [document in Mixpanel](https://developer.mixpanel.com/reference/import-events#propertiesinsert_id). Otherwise, this will result in 400 Bad Request error from Mixpanel.
@tanjinhong72 tanjinhong72 marked this pull request as ready for review September 19, 2024 01:07
@tanjinhong72 tanjinhong72 requested a review from a team as a code owner September 19, 2024 01:07
@forstisabella forstisabella merged commit ae58965 into develop Sep 19, 2024
4 checks passed
@forstisabella forstisabella deleted the tanjinhong72-patch-3 branch September 19, 2024 17:17
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants