Skip to content

Mantain customer subscription to the newsletter#98

Open
idnovate wants to merge 2 commits intoPrestaShop:devfrom
idnovate:patch-1
Open

Mantain customer subscription to the newsletter#98
idnovate wants to merge 2 commits intoPrestaShop:devfrom
idnovate:patch-1

Conversation

@idnovate
Copy link

Questions Answers
Description? Check issue
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? Fixes PrestaShop/ps_emailsubscription#34035
How to test? Check issue

Copy link
Contributor

@Hlavtox Hlavtox left a comment

Choose a reason for hiding this comment

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

I agree with the change, but I think it should be configurable.

I mean -> if the user makes an account WITHOUT that checkbox checked, doesn't it mean that he doesn't want to receive a newsletter? ;-)

I know that @MatShir validated the issue, but I would like to have a second thought of @PrestaShop/product-council. :-)

@idnovate
Copy link
Author

I agree with the change, but I think it should be configurable.

I mean -> if the user makes an account WITHOUT that checkbox checked, doesn't it mean that he doesn't want to receive a newsletter? ;-)

I know that @MatShir validated the issue, but I would like to have a second thought of @PrestaShop/product-council. :-)

The point is that he already got subscribed to the newsletter, and he didn't ask for the unsubscription.

@Hlavtox
Copy link
Contributor

Hlavtox commented Sep 26, 2023

Yessss, but he could have subscribed 2 months ago and doesn't even remember it.

@idnovate
Copy link
Author

idnovate commented Sep 26, 2023

I understand.

Another point would be getting this information when the customer has introduced the email, and checking the input if he is subscribed.

So he could opt out before registering.

@MatShir
Copy link

MatShir commented Sep 26, 2023

Or hide the check? If he's already a subscriber, should we encourage him to leave? He might uncheck it by reflexe

@RosaBenouamer
Copy link

Or check the newsletter by default ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for review

Development

Successfully merging this pull request may close these issues.

Customer subscription lost after registration

4 participants