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

Explore enabling the payment method by default #71

Closed
seand7565 opened this issue Jul 21, 2020 · 1 comment · Fixed by #77
Closed

Explore enabling the payment method by default #71

seand7565 opened this issue Jul 21, 2020 · 1 comment · Fixed by #77
Assignees
Labels
paypal onboarding Related to Merchant Onboarding and account setup solidus admin Touches the Solidus backend/admin

Comments

@seand7565
Copy link
Contributor

We've got the payment method disabled by default - because we want to ensure the users email is confirmed before proceeding. The current flow is:
Create payment method -> Set "PayPal Email Confirmed" to true -> Set "Available to Users" to true

Which is a bit unwieldy. I think enabling the payment method by default is fine, but then the "PayPal Email Confirmed" check is useless.

From Hai -

In the admin panel, the admin/merchant has to click “Available to users” — will this be enabled by default? I believe it should be to make the setup/onboarding process smoother for merchants.

@seand7565 seand7565 added solidus admin Touches the Solidus backend/admin paypal onboarding Related to Merchant Onboarding and account setup labels Jul 21, 2020
@elia
Copy link
Member

elia commented Jul 22, 2020

We could turn the check into an alert box of some kind that reminds the users to confirm the email address, optionally we can include a link (target=_blank) to where the confirmation is supposed to happen, or to a page that allows resending it, or to some PP documentation.

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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants