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

Upgrade savon httpi to fix integration dependency errors #18

Merged
merged 92 commits into from
Jun 26, 2024

Conversation

banjoko01
Copy link

This PR:

  • Update the original repo here with the forked repo, whilst keeping all custom changes made.
  • Updates savon, httpi and wasabi gem version

Jacob Burkhart & Andy Delcambre and others added 30 commits June 7, 2012 12:54
Support proper config object for authentication
…ialized object when creating a SubscribeRequest.
…ly materialized object when creating a SubscribeRequest."

This reverts commit eb66903.
Support multiple rate plans in a subscription request.

Conflicts:
	lib/zuora/objects/subscribe_request.rb
@banjoko01 banjoko01 marked this pull request as ready for review June 21, 2024 11:08
Copy link

@ianoxley ianoxley left a comment

Choose a reason for hiding this comment

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

There's a lot of commented out code that either was commented out and now isn't, or wasn't commented out and now is. Have all these changes come from upstream?

lib/zuora/objects/invoice.rb Outdated Show resolved Hide resolved
lib/zuora/objects/invoice_adjustment.rb Outdated Show resolved Hide resolved
lib/zuora/objects/invoice_item.rb Outdated Show resolved Hide resolved
lib/zuora/objects/invoice_item_adjustment.rb Outdated Show resolved Hide resolved
lib/zuora/objects/subscription.rb Outdated Show resolved Hide resolved
lib/zuora/objects/usage.rb Outdated Show resolved Hide resolved
lib/zuora/api.rb Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
- Update repo to clone from in `README.md`
- Update test accordingly
- Update validations for attributes with changed datatype. `(i.e Datetime to date)`
@banjoko01 banjoko01 requested review from ianoxley and tapan-sh June 25, 2024 12:19
lib/zuora/api.rb Outdated Show resolved Hide resolved
Copy link

@ianoxley ianoxley left a comment

Choose a reason for hiding this comment

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

If we can add those how to run specs with appraisal instructions to the README too that'd be great.

lib/zuora/api.rb Outdated Show resolved Hide resolved
lib/zuora/api.rb Outdated Show resolved Hide resolved
lib/zuora/objects/refund.rb Outdated Show resolved Hide resolved
lib/zuora/objects/payment_method.rb Show resolved Hide resolved
@banjoko01 banjoko01 requested a review from ianoxley June 26, 2024 11:01
ianoxley
ianoxley previously approved these changes Jun 26, 2024
Copy link

@ianoxley ianoxley left a comment

Choose a reason for hiding this comment

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

Just some minor comments on the README changes

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
ianoxley
ianoxley previously approved these changes Jun 26, 2024
ianoxley
ianoxley previously approved these changes Jun 26, 2024
zuora.gemspec Outdated Show resolved Hide resolved
@banjoko01 banjoko01 merged commit 25ab50c into master Jun 26, 2024
6 checks passed
@adamgeorgeson adamgeorgeson deleted the upgrade-savon-httpi branch July 1, 2024 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.