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

DO NOT MERGE - Bring Stripe Connect tweaks in from 2-0-stable-stripe #3

Open
wants to merge 22 commits into
base: 2-2-stable
Choose a base branch
from

Conversation

glongman
Copy link

No description provided.

@glongman glongman changed the title 2 2 stable stripe DO NOT MERGE - Bring Stripe Connect tweaks in from 2-0-stable-stripe Jun 18, 2015
glongman and others added 16 commits July 8, 2015 10:57
remove attr_accessible from StripeGateway
Removed referenced issue as it was implemented upstream.
Populating credit card fields from activemerchant response
- Using an environment variable to access the publishable key
- Using different jQuery anchors
- Using an environment variable to access the publishable key
- Using different jQuery anchors
StripeJS support for Spree admin and patient checkout
The click handler from Spree  didn't handle the case if "New Payment"
wasn't selected. Now the click handler will do a plain submit on the
form if anything else is selected.
Ensure application_fee ends up in the purchase options
@uri
Copy link

uri commented Mar 12, 2016

Is the intention that we port this into 2-2-stable and then rebase our branch 2-2-stable-healthwave off of it?

@@ -73,8 +86,11 @@ def options

def options_for_purchase_or_auth(money, creditcard, gateway_options)
options = {}
options[:description] = "Spree Order ID: #{gateway_options[:order_id]}"
options[:description] = "HealthWave Order: #{gateway_options[:order_id]}"

Choose a reason for hiding this comment

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

Should this be updated?

Copy link
Author

Choose a reason for hiding this comment

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

prbly

Copy link
Author

Choose a reason for hiding this comment

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

whoa - fyi - this PR predates the work Uri did bringing in tokens.

kurtfunai and others added 4 commits December 14, 2016 14:44
Transaction description to read Fullscript instead of HealthWave Order
…-version

Pass through specific stripe api version to activemerchant gem
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.

4 participants