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

[Stripe] does not use Money protocol #109

Closed
1 of 2 tasks
oyeb opened this issue Jan 25, 2018 · 1 comment
Closed
1 of 2 tasks

[Stripe] does not use Money protocol #109

oyeb opened this issue Jan 25, 2018 · 1 comment
Assignees
@chandradot99
Labels
enhancement hotlist: core-team severity: medium
Milestone
1.1.x

Comments

@oyeb
Copy link
Contributor

oyeb commented Jan 25, 2018 •
edited

  • Use the Money protocol
  • Update test cases and examples
@oyeb oyeb added this to the 1.1.0-beta milestone Jan 25, 2018
@oyeb oyeb mentioned this issue Jan 30, 2018
Merged
oyeb added a commit that referenced this issue Mar 20, 2018
@oyeb
Adapts Stripe with the money protocol
c7777e3 
* Fixes #5 and #109
* Moved the stripe_test to integration.
* Fixed credo issue in money integration test

Changes to MONEI
* Removed unnecessary clauses from MONEI
* Re-formatted source.
ashish173 pushed a commit that referenced this issue Mar 21, 2018
@oyeb @ashish173
Adapts Stripe with the money protocol
a9b76db 
* Fixes #5 and #109
* Moved the stripe_test to integration.
* Fixed credo issue in money integration test

Changes to MONEI
* Removed unnecessary clauses from MONEI
* Re-formatted source.
ashish173 pushed a commit that referenced this issue Apr 22, 2018
@oyeb @ashish173
Adapts Stripe with the money protocol
10f5058 
* Fixes #5 and #109
* Moved the stripe_test to integration.
* Fixed credo issue in money integration test

Changes to MONEI
* Removed unnecessary clauses from MONEI
* Re-formatted source.
ashish173 pushed a commit that referenced this issue Apr 22, 2018
@oyeb @ashish173
Adapts Stripe with the money protocol
921853c 
* Fixes #5 and #109
* Moved the stripe_test to integration.
* Fixed credo issue in money integration test

Changes to MONEI
* Removed unnecessary clauses from MONEI
* Re-formatted source.
@oyeb
Copy link
Contributor Author

oyeb commented Jun 4, 2018

Closing as this is fixed in #115

@oyeb oyeb closed this as completed Jun 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chandradot99 chandradot99

Labels
enhancement hotlist: core-team severity: medium
Projects
None yet
Milestone
1.1.x
Development

No branches or pull requests
2 participants
@oyeb @chandradot99