-
Notifications
You must be signed in to change notification settings - Fork 93
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
Bump to 2.10.4 / API version 2.14 #363
Conversation
ddf07d5
to
dc3f103
Compare
Tests/Recurly/Subscription_Test.php
Outdated
@@ -240,6 +240,11 @@ public function testUpdateNotes() { | |||
|
|||
$notes = array("customer_notes" => "New Customer Notes", "terms_and_condititions" => "New Terms", "vat_reverse_charge_notes" => "New VAT Notes"); | |||
|
|||
// You can also update custom fields through this endpoint | |||
$cf = $subscription->custom_fields->offsetGet("shasta"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit pick but I think the preferred style for this would be to just use the array operator:
$subscription->custom_fields["shasta"];
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, I think that's worth fixing. I'll push up a commit on top.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I actually didn't know that the bracket syntax worked. TIL, thanks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This release will upgrade us to API version 2.14. There are no breaking changes.