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

Cleanup or option to manually process failed 'immediate' items? #252

Closed
davinian opened this issue Oct 29, 2024 · 1 comment
Closed

Cleanup or option to manually process failed 'immediate' items? #252

davinian opened this issue Oct 29, 2024 · 1 comment

Comments

@davinian
Copy link

Hi, I've a support thread open on wp.org regarding stuck emails in the wp_bpges_queued_items table (https://wordpress.org/support/topic/group-emails-stopped-after-wordpress-release-6-5-4-june-5-2024-2/).

While looking at other support questions/answers I found a link to the file classes/class-bpges-async-request-send-queue.php and noticed a // @todo for possibly cleaning up failed 'immediate' items?

I would vote thumbs up for this feature, but maybe it could have an option in the plugin settings to purge failed items after X days?

I would also be interested to know if it's possible to try resending failed items? Perhaps something similar to above where there's a setting to try and resend failed items up to X hours or maybe days after they failed?

@boonebgorges
Copy link
Owner

See #163. That ticket proposes an automated cleanup routine. Having a manual tool feels worse to me, since (1) this should theoretically not happen, (2) if it does happen, it's better to try to send the emails rather than delete them, (3) it doesn't actually hurt anything if the queued items remain in the DB indefinitely.

I would also be interested to know if it's possible to try resending failed items?

The automatic resending would probably be part of the health-check proposed in #163. As for a manual technique, I'm wary of putting one in the plugin, but I posted on the wordpress.org thread with a technique you could probably use.

I'm going to close this ticket, but please feel free to follow and share ideas on #163.

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

2 participants