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

List size issue -- it's not automatically removing users #606

Open
leobarcellos opened this issue Jan 6, 2025 · 4 comments
Open

List size issue -- it's not automatically removing users #606

leobarcellos opened this issue Jan 6, 2025 · 4 comments

Comments

@leobarcellos
Copy link
Contributor

We've noticed that list rules are not removing users, they are only adding more users.

Example:

We have lists of people that opened or clicked an email on last 5 days (we are using dynamic dates, using the handlebars)
If let "alone", list just keep increasing through days.

However when we enter on the list and save it, it correctly updates it with the real size (removing who don't belong to it).

Maybe related to #538 or #566

@pushchris
Copy link
Contributor

@leobarcellos can you confirm what the list size is in the database for some of these? Would be helpful to know if the total number of rows in the database matches the count or if the count is deviating over time

@tanmay-predisai
Copy link

We also see a similar issue.

The list size as shown on the UI is 3X the size when checked in MySQL using select count(*) from user_list where list_id = 22 and deleted_at is NULL;

I had regenerated the list yesterday and it started showing 3X size on the UI post that.

@leobarcellos
Copy link
Contributor Author

@leobarcellos can you confirm what the list size is in the database for some of these? Would be helpful to know if the total number of rows in the database matches the count or if the count is deviating over time

On UI it's showing more than on database, I think it keeps getting worse through time.

Example:
Screenshot 2025-01-14 at 12 16 37 PM
Screenshot 2025-01-14 at 12 16 29 PM

This one is kinda close, but I've seen worse

@leobarcellos
Copy link
Contributor Author

I've added the last version you merged that addresses this issue, thank you @pushchris .

I will keep watching and I either update here or close the issue.

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