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

Fixed an email pending count error #722

Merged
merged 1 commit into from Aug 19, 2015

Conversation

@alanhartless
Copy link
Contributor

commented Jul 27, 2015

Description
The pending count was off for list based emails if the leads had received any other email. This was due to a query expression that wasn't actually getting added to the query.

Testing
Setup two list based emails assigned to the same lead group and send the first email to the leads. Now the second email will have 0. Apply the PR and the correct Pending count should be displayed.

…ther email had already been sent to the leads
@alanhartless alanhartless added this to the 1.1.4 milestone Jul 27, 2015
@dbhurley

This comment has been minimized.

Copy link
Member

commented Aug 2, 2015

+1 works to properly show pending count.

@cuberoot

This comment has been minimized.

Copy link

commented Aug 6, 2015

FYI, this fixed the same issue for me.

@escopecz

This comment has been minimized.

Copy link
Member

commented Aug 19, 2015

Yup, this fixes it 👍

escopecz added a commit that referenced this pull request Aug 19, 2015
@escopecz escopecz merged commit a2107f5 into mautic:staging Aug 19, 2015
1 check passed
1 check passed
Scrutinizer 10 new issues
Details
@alanhartless alanhartless deleted the alanhartless:pending-count-fix branch Aug 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.