Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

Let PayPal button to receive locale/style parameters #236

Merged
merged 3 commits into from
Oct 9, 2019

Conversation

MinasMazar
Copy link
Contributor

@MinasMazar MinasMazar commented Sep 20, 2019

ref #229

Default values of Paypal button style should be the most suitable to current Solidus UX style.

screenshot-localhost_3000-2019 09 27-16_20_08

@MinasMazar MinasMazar force-pushed the mm/i229-paypal-button-style branch 4 times, most recently from 01cee72 to 9dc8e8b Compare September 27, 2019 13:32
@MinasMazar MinasMazar force-pushed the mm/i229-paypal-button-style branch from 9dc8e8b to a384d55 Compare September 27, 2019 13:52
Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

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

Thanks, Flavio!

@MinasMazar MinasMazar force-pushed the mm/i229-paypal-button-style branch 3 times, most recently from c41d69f to de8cb94 Compare September 27, 2019 16:12
@MinasMazar MinasMazar force-pushed the mm/i229-paypal-button-style branch from de8cb94 to 673e50d Compare September 27, 2019 16:29
Copy link
Member

@tvdeyen tvdeyen left a comment

Choose a reason for hiding this comment

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

Thank you!

@tvdeyen tvdeyen merged commit c21f870 into solidusio:master Oct 9, 2019
@MinasMazar MinasMazar deleted the mm/i229-paypal-button-style branch October 18, 2019 14:36
MinasMazar pushed a commit to nebulab/solidus_paypal_braintree that referenced this pull request Oct 18, 2019
This PR solidusio#236
was incomplete since adds PayPal button customizable parameters only for
checkout/payment step. This PR adds the same feature also for
cart page.
MinasMazar pushed a commit to nebulab/solidus_paypal_braintree that referenced this pull request Oct 18, 2019
This PR solidusio#236
was incomplete since adds PayPal button customizable parameters only for
checkout/payment step. This PR adds the same feature also for
cart page.
MinasMazar pushed a commit to nebulab/solidus_paypal_braintree that referenced this pull request Nov 15, 2019
This PR solidusio#236
was incomplete since adds PayPal button customizable parameters only for
checkout/payment step. This PR adds the same feature also for
cart page.
MinasMazar pushed a commit to nebulab/solidus_paypal_braintree that referenced this pull request Nov 15, 2019
This PR solidusio#236
was incomplete since adds PayPal button customizable parameters only for
checkout/payment step. This PR adds the same feature also for
cart page.

Also add an ad-hoc example at cart page feautre test
MinasMazar pushed a commit to nebulab/solidus_paypal_braintree that referenced this pull request Nov 15, 2019
This PR solidusio#236
was incomplete since adds PayPal button customizable parameters only for
checkout/payment step. This PR adds the same feature also for
cart page.

Also add an ad-hoc example at cart page feautre test
MinasMazar pushed a commit to nebulab/solidus_paypal_braintree that referenced this pull request Nov 15, 2019
This complete solidusio#229 since solidusio#236 was incomplete: adds PayPal button customizable
parameters only for checkout/payment step. This PR adds the same feature also
for cart page.

Ref. solidusio#229
Ref. solidusio#236
MinasMazar pushed a commit to nebulab/solidus_paypal_braintree that referenced this pull request Nov 15, 2019
This complete solidusio#229 since solidusio#236 was incomplete: adds PayPal button customizable
parameters only for checkout/payment step. This PR adds the same feature also
for cart page.

Ref. solidusio#229
Ref. solidusio#236
MinasMazar pushed a commit to nebulab/solidus_paypal_braintree that referenced this pull request Oct 9, 2020
Ref. solidusio#251

Using `Spree::CheckoutController.helper` and
`Spree::OrdersController.helper` in `config/initializers/braintree.rb`
to include helper modules does not work: at `checkout/payement` step an
exception is raised

```
ActionView::Template::Error (undefined method `paypal_button_preference`)
```

This commit move `paypal_button_preferences` (introduced here[1]) in the already
present `SolidusPaypalBraintree::BraintreeCheckoutHelper` module.

[1] solidusio#236
MinasMazar pushed a commit to nebulab/solidus_paypal_braintree that referenced this pull request Oct 15, 2020
Ref. solidusio#251

Using `Spree::CheckoutController.helper` and
`Spree::OrdersController.helper` in `config/initializers/braintree.rb`
to include helper modules does not work: at `checkout/payement` step an
exception is raised

```
ActionView::Template::Error (undefined method `paypal_button_preference`)
```

This commit move `paypal_button_preferences` (introduced with [1]) in
the already present `SolidusPaypalBraintree::BraintreeCheckoutHelper`
module, and adds the same helper method to the
`Spree::OrdersController` (in the same way done here [2]) to enable
the button also in the cart page.

[1] solidusio#236
[2] solidusio@28d0de9
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants