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

Issue #3490695: Wrong users number on bulk email sending #4220

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

nechai
Copy link
Contributor

@nechai nechai commented Nov 29, 2024

Problem

In #3905 we added the ability to prevent sending emails on bulk operation for groups and events if a user is unsubscribed from these email receiving.

On the group "manage members" page, the group manager can send an email collectively to selected users.

The problem is that if the group has a large number of members (> 100), then choosing the "select all" option, only a part of users will receive the email (this part is always equal to the number of pager).

Solution

If select all users and send bulk email:
Screenshot 2024-11-29 at 18 23 15

Before:
Screenshot 2024-11-29 at 18 27 58

After:
Screenshot 2024-11-29 at 18 23 41

Before:
Screenshot 2024-11-29 at 18 29 38

After:
Screenshot 2024-11-29 at 18 24 14

Release notes

Fix the wrong users number during bulk email sending

Issue tracker

How to test

  • Login as admin
  • Enable modules: social_group_flexible_group, social_group_gvbo, social_email_broadcast
  • Create a flexible group
  • Add more than 100 members to a group
  • Try to send bulk email to all users on /group/{gid}/membership
    -->

@nechai nechai added type: bug Fixes a bug in Open Social team: enterprise This PR originates from the ECI team status: needs review This pull request is waiting for a requested review prio: medium labels Nov 29, 2024
@nechai nechai added this to the 13.0.0-alpha19 milestone Nov 29, 2024
@nechai nechai self-assigned this Nov 29, 2024
@open-social-tugboat
Copy link

Tugboat has finished building the preview for this pull request!

Link:

Dashboard:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio: medium status: needs review This pull request is waiting for a requested review team: enterprise This PR originates from the ECI team type: bug Fixes a bug in Open Social
Development

Successfully merging this pull request may close these issues.

2 participants