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] Adapted for money protocol #115

Merged
merged 1 commit into from
Mar 21, 2018
Merged

[Stripe] Adapted for money protocol #115

merged 1 commit into from
Mar 21, 2018

Conversation

oyeb
Copy link
Contributor

@oyeb oyeb commented Jan 30, 2018 •
edited
Loading

@oyeb oyeb added this to the 1.1.0-beta milestone Jan 30, 2018
@oyeb oyeb self-assigned this Jan 30, 2018
@oyeb oyeb requested review from pkrawat1 and chandradot99 January 30, 2018 10:55
@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 ashish173 merged commit a9b76db into dev Mar 21, 2018
@ashish173 ashish173 deleted the stripe-money branch March 21, 2018 10:29
@oyeb oyeb mentioned this pull request Jun 4, 2018
Closed
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chandradot99 chandradot99 chandradot99 approved these changes

@pkrawat1 pkrawat1 Awaiting requested review from pkrawat1

Assignees

@oyeb oyeb

Labels
enhancement
Projects
None yet
Milestone
1.1.x
Development

Successfully merging this pull request may close these issues.
3 participants
@oyeb @chandradot99 @ashish173