Skip to content

Include dot notation limitation #5038

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

Merged
merged 3 commits into from
Jul 27, 2023
Merged

Include dot notation limitation #5038

merged 3 commits into from
Jul 27, 2023

Conversation

senabon
Copy link
Contributor

@senabon senabon commented Jul 19, 2023

Actions Mappings don't support dot notation in mapping fields

Proposed changes

Several customers are sending in payloads with dot notation and then trying to map them but are unsuccessful because dot notation isn't supported.

Merge timing

  • ASAP once approved

Related issues (optional)

https://segment.zendesk.com/agent/tickets/516943
https://twilio.slack.com/archives/CC97A542H/p1681243270312659

Actions Mappings don't support dot notation in mapping fields
@senabon senabon requested a review from a team as a code owner July 19, 2023 18:12
@senabon senabon requested review from stayseesong and removed request for a team July 19, 2023 18:12
@stayseesong stayseesong added the content-update updates to content that are not new features, includes grammar fixes, added notes label Jul 27, 2023
@stayseesong stayseesong merged commit d074658 into develop Jul 27, 2023
@stayseesong stayseesong deleted the senabon-patch-3 branch July 27, 2023 16:16
@github-actions
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
content-update updates to content that are not new features, includes grammar fixes, added notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants