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

Should we enforce email address verification before allowing the payment method to be usable? #7

Closed
seand7565 opened this issue May 18, 2020 · 2 comments
Labels
paypal onboarding Related to Merchant Onboarding and account setup question Further information is requested solidus admin Touches the Solidus backend/admin

Comments

@seand7565
Copy link
Contributor

PayPal wants us to check the users email verification through an API endpoint before processing transactions. This seems like something that the user should verify on their own, but we do have an API endpoint for checking their email verification status, so we could prevent the payment method from being available to users before the email is verified?

@seand7565 seand7565 added question Further information is requested solidus admin Touches the Solidus backend/admin labels May 18, 2020
@elia elia added this to To do in Roadmap & Progress May 21, 2020
@elia elia added the paypal onboarding Related to Merchant Onboarding and account setup label May 21, 2020
@elia
Copy link
Member

elia commented May 21, 2020

I think would be "nice to have", I'll open an issue for implementing this, and we can keep it at a lower priority.

@elia
Copy link
Member

elia commented May 21, 2020

Closing in favor of #9

@elia elia closed this as completed May 21, 2020
Roadmap & Progress automation moved this from To do to Done May 21, 2020
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 question Further information is requested solidus admin Touches the Solidus backend/admin
Projects
No open projects
Development

No branches or pull requests

2 participants