-
Notifications
You must be signed in to change notification settings - Fork 783
Exclude sections from camel/snake case conversion in Mail helper class #687
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
Conversation
Thanks @simonbelanger! How did you figure out |
It wasn't working so I took a look at the payload sent to the API by the package and saw that the sections delimiters were all messed up. |
Nice! Thanks! @adamreisnz LGTM. LGTY? Happy Friday! |
Awesome, thanks for checking and fixing this! |
Do you have an ETA for the deployment of this fix? |
I don't have an ETA just yet, but additional +1's and comments do help issues gain priority in our backlog. |
@thinkingserious this can probably be merged safely, as it's essentially a bug fix, for a patch release. |
Thanks @adamreisnz! I've got this on the backlog for a release. |
Hello @simonbelanger, |
@thinkingserious My pleasure! Form filled. |
No description provided.