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.

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

Add RevRec POBs support #792

Merged
merged 1 commit into from
Feb 6, 2024

Conversation

faiberrec
Copy link

@faiberrec faiberrec commented Feb 2, 2024

This change creates the PerformanceObligation entity for the V2 client, part of the new (and upcoming) RevRec API features.

@faiberrec faiberrec added the V2 V2 Client label Feb 2, 2024
$this->assertInstanceOf('Recurly_PerformanceObligation', $pob);
$this->assertEquals('Over Time (Daily)', $pob->name);
$this->assertEquals('6', $pob->id);
$this->assertInstanceOf('DateTime', $pob->created_at);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any particular reason you omitted testing for updated_at? 🤔

Copy link

@Smolations Smolations left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to add a dev dependency of fakerphp/faker (^1.23.0) in the composer.json file for testing purposes.

@tilley-kyle
Copy link
Contributor

DevQA+1

@tilley-kyle tilley-kyle merged commit 96c9ec1 into add_revrec_support_to_php Feb 6, 2024
10 checks passed
@tilley-kyle tilley-kyle deleted the add_performance_obligations branch February 6, 2024 21:46
recurly-integrations pushed a commit that referenced this pull request Mar 13, 2024
[Full Changelog](2.14.7...2.14.8)

**Merged Pull Requests**

- Add RevRec to adjustments/purchases [#802](#802) ([faiberrec](https://github.com/faiberrec))
- Add revrec attributes to add ons [#801](#801) ([faiberrec](https://github.com/faiberrec))
- Add revrec attributes to items and giftcards [#798](#798) ([faiberrec](https://github.com/faiberrec))
- Add RevRec attributes to shipping methods [#796](#796) ([faiberrec](https://github.com/faiberrec))
- Add revrec attributes to plans [#795](#795) ([faiberrec](https://github.com/faiberrec))
- Add default glas to Business Entities [#794](#794) ([faiberrec](https://github.com/faiberrec))
- Add revrec support to php [#793](#793) ([faiberrec](https://github.com/faiberrec))
- Add RevRec POBs support [#792](#792) ([faiberrec](https://github.com/faiberrec))
- Add GeneralLdegerAccounts(glas) resources [#791](#791) ([faiberrec](https://github.com/faiberrec))
- add external payment phases [#784](#784) ([amandamfielding](https://github.com/amandamfielding))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
V2 V2 Client
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants