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

Re-subscription confirmation glitch #1638

Closed
javierarce opened this issue Dec 23, 2023 · 1 comment
Closed

Re-subscription confirmation glitch #1638

javierarce opened this issue Dec 23, 2023 · 1 comment

Comments

@javierarce
Copy link

Version:

  • listmonk: 2.5.1
  • OS: Ubuntu 20.04.6 LTS

Steps to Reproduce This Issue:

  1. Create a public double opt-in list and subscribe to it.
  2. Confirm the subscription from the opt-in email.
  3. Send a campaign.
  4. Unsubscribe from the list through the campaign email.

At this point, the user is unsubscribed but still exists in the listmonk db without subscriptions.

  1. Subscribe again to the same list.

Expected Result:

  • The user sees a screen indicating that they need to confirm the subscription.
  • The user receives an opt-in message to confirm the subscription.

Actual Result:

  • The user sees a screen indicating that they were successfully subscribed to the list.
  • In the admin panel, the user is not subscribed.

This error is similar to this issue. The difference in this case is that if a user is unsubscribed from a list and tries to subscribe again, they'll see a message indicating that the subscription was successful, but they won't be subscribed. Adding to the confusion, they'll get a message to opt-in. If they confirm the subscription, they'll get correctly subscribed, but obviously the previous message is confusing.

@javierarce javierarce added the bug Something isn't working label Dec 23, 2023
@javierarce
Copy link
Author

I suspect the problem lies in this line of the processSubForm method. If the subscriber already exists, the method returns false no matter what. I think the method should check the subscription status to return true or false accordingly.

@knadh knadh added minor-bug and removed bug Something isn't working labels Dec 30, 2023
@knadh knadh closed this as completed in b2a5e37 Dec 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants