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.

Sign up for GitHub

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

Jump to bottom

Minor changes to match the current API version #10

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Minor changes to match the current API version #10

wants to merge 8 commits into from

Conversation

dstdia
Copy link

@dstdia dstdia commented Feb 6, 2016

  • Adds some improvements to the StripeEvent and StripeCharge classes
  • changed callouts to match current API
  • addressed issues with the custom objects included in the package

Still TODO:

  • re-creating test data from API documentation
  • overall check if classes still match current API definition

@dstdia
Added missing fields that were referenced by code to the Stripe_Webho…
55585f8 
…ok__c-object
@dstdia
Some refurbishments to adapt to current API
c2e3a69 
Make some minor adjustments to adapt to the current API.
TODO:
- Clean up / modernize test data.
- Check API compatibility for all components
@dstdia
Improvements to Charges and Events to match the current API version
a9ca799
@dstdia
Merge branch 'master' into diadst
018a241
@dstdia
Update README.md
abdf7d9
@dstdia
Update README.md
bd0cb67
@dstdia
Update README.md
dfff5e2
@dstdia
Added runAs() to test
ce28f0c 
Test data bound the successful execution of StripeUtilTests to the timezone of the running user. 

David Boukhors contributed a patch. Thank you, David!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dstdia @davidboukhors