-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add Braintree to the installer as a payment method option #4961
Add Braintree to the installer as a payment method option #4961
Conversation
123947d
to
e3f7cf1
Compare
e3f7cf1
to
f274a66
Compare
@@ -0,0 +1,5 @@ | |||
unless Bundler.locked_gems.dependencies['solidus_braintree'] | |||
bundle_command "add solidus_braintree --git='https://github.com/solidusio/solidus_braintree' --branch='master'" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The git and branch options are temporary. Once SolidusBraintree 3 is released, we can change this to "add solidus_braintree ~> 3.0"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @gsmendoza. Nice work. I left some suggestions. Also, holding approval until we release 2.0.
f274a66
to
e4ed928
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @gsmendoza! Everything looks fine. Holding approval until we release the new version of the gem.
e4ed928
to
4df1ae7
Compare
4df1ae7
to
3b9d196
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a much needed cleanup! π
It should be applied equally to all enum options like payment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π
Summary
Closes #4749.
Blockers
This is dependent on releasing SolidusBraintree v3 (SolidusBraintree with StarterFrontend generator). See solidusio/solidus_braintree#94.
Code review feedback
Checklist
Check out our PR guidelines for more details.
The following are mandatory for all PRs:
The following are not always needed: