Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Bugfix: Change "enabled" to "enable" to conform to Sendgrid API. (#624)
# Problem [A recent PR](#618) added click tracking support for the SendGrid adapter. Unfortunately, we didn't catch a typo that sends "enabled" as the param instead of "enable" as is recognized by the API. This results in the following error: ``` "{\"errors\":[{\"message\":\"The click_tracking enable parameter is required.\",\"field\":\"tracking_settings.click_tracking.enable\",\"help\":\"http://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/errors.html#message.tracking_settings.click_tracking.enable\"}]}" ``` with these params: ``` "tracking_settings" => %{"click_tracking" => %{"enable_text" => false, "enabled" => false} ``` Where it says `"enabled" => false` in the params, it should say `"enable" => false` # Solution Fix the typo and update the tests.
- Loading branch information