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

Add ability for admin users to customize paypal button #16

Merged
merged 1 commit into from
Jun 8, 2020

Conversation

seand7565
Copy link
Contributor

I utilize the preference_fields, creating a custom one for a select option. I also provide defaults for
all of the button options based on the defaults PayPal recommends selecting, while adding a helper
that provides the collection of other available options. All of this is displayed on the PCP
payment method page on admin. The options are passed on the frontend when we display the PayPal button.

NOTE: This PR requires #10 to be merged first, as it adds some options to the frontend button that that PR adds. Once #10 is merged, I'll rebase this PR and it should be good to go!

Ref: #5

Copy link
Member

@elia elia left a comment

Choose a reason for hiding this comment

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

just a couple of comments

@seand7565 seand7565 force-pushed the customize_buttons branch from 8d7f153 to e270f99 Compare June 1, 2020 17:38
Copy link
Member

@elia elia left a comment

Choose a reason for hiding this comment

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

LGTM! Pending a rebase of course 👍

@elia elia assigned elia and seand7565 and unassigned elia Jun 4, 2020
@elia elia added paypal onboarding Related to Merchant Onboarding and account setup solidus admin Touches the Solidus backend/admin solidus frontend Touches the Solidus frontend/storefront labels Jun 4, 2020
I utilize the preference_fields, creating a custom one for a select option. I also provide defaults for
all of the button options based on the defaults PayPal recommends selecting, while adding a helper
that provides the collection of other available options. All of this is displayed on the PCP
payment method page on admin. The options are passed on the frontend when we display the PayPal button.
@seand7565 seand7565 force-pushed the customize_buttons branch from e270f99 to 21316be Compare June 8, 2020 11:51
@elia elia merged commit 2805787 into solidusio:master Jun 8, 2020
@elia elia deleted the customize_buttons branch June 8, 2020 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
paypal onboarding Related to Merchant Onboarding and account setup solidus admin Touches the Solidus backend/admin solidus frontend Touches the Solidus frontend/storefront
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants