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

v158: Temporary customer account created by paypalwpp not removed #5058

Open
lat9 opened this issue Jul 26, 2022 · 4 comments
Open

v158: Temporary customer account created by paypalwpp not removed #5058

lat9 opened this issue Jul 26, 2022 · 4 comments

Comments

@lat9
Copy link
Contributor

lat9 commented Jul 26, 2022

If the PayPal Express Checkout payment-module is enabled, with its "Express Checkout Button" enabled and "Express Checkout: Automatic Account Creation" set to 'no', a temporary account is created for the customer when paying via a click on the PPEC button if they don't already have one. That account is supposed to be removed after order completion.

If the customer (for whatever reason) abandons the order, that account is still present in the store.

@lat9 lat9 changed the title v158: Temporary customer account not removed if order abandoned v158: Temporary customer account created by paypalwpp not removed Jul 27, 2022
@lat9
Copy link
Contributor Author

lat9 commented Jul 27, 2022

Noting that the supposedly temporary account isn't removed after a successful checkout, either.

@scottcwilson
Copy link
Sponsor Contributor

That account is supposed to be removed after order completion.

Is this documented (and are we sure it's not an error in the documentation)?

It seems like a change that would have limited benefit - are we sure we want to pursue?

@lat9
Copy link
Contributor Author

lat9 commented Jul 15, 2023

It'll be confusing to the customer if they come back to create an account, since the email-address will be denied.

@drbyte
Copy link
Member

drbyte commented Jul 28, 2023

One challenge is determining "how long does 'temporary' mean?"

Also, if the account is temporary, could we just exclude lookups in the Customer::doLoginLookupByEmail() function?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants