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

potential fix for #76 #97

Closed
wants to merge 1 commit into from
Closed

Conversation

bjendres
Copy link

Potential "hotfix" for #76.
Someone would have to look over this, though. I have no idea what it was meant to do in the first place.

@drastik
Copy link
Owner

drastik commented Sep 10, 2015

In the code's comment:
// Handle multiple payment options and Stripe not being chosen.

It was a another check in place to make sure Stripe doesn't get involved if there are multiple payment methods enabled on the site, and Stripe wasn't the actively chosen one.
Looks like 4.6 has changed some things on us and need to revisit our options.

@andrea-f
Copy link

andrea-f commented Oct 1, 2015

Is there any news with this?

@bjendres
Copy link
Author

bjendres commented Oct 1, 2015

No from our end, sorry. It worked for us.

Follow #76 for current developments.

@bjendres bjendres closed this Oct 1, 2015
@andrea-f
Copy link

andrea-f commented Oct 3, 2015

Yes thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants