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.

Sign up for GitHub

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

Jump to bottom

Support nested map params (eg. Notification data) #171

Open
superchris opened this issue Feb 18, 2022 · 1 comment
Open

Support nested map params (eg. Notification data) #171

superchris opened this issue Feb 18, 2022 · 1 comment
Labels
enhancement help wanted

Comments

@superchris
Copy link

Currently, it appears there is no support for nested maps in Twilio API calls. I first encountered this attempted to call ExTwilio.Notify.Notifications.create with a data property that is a map. It appears that ExTwilio.Api is not able to handle this. It seems possible that changing it to use JSON as the content-type instead of application/x-www-form-urlencoded would solve problems like this. I'd be happy to have a go at doing this, if there is not a reason this is a Bad Idea.
@superchris
Copy link
Author

Hmm.. obviously not as simple as that. All the params are capitalized and camel cased. Thanks, Twilio. So gross....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@superchris @danielberkompas