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

Auto ForceDelete on email bounces/verification fails #1874

Conversation

OlegPhenomenon
Copy link
Contributor

@OlegPhenomenon OlegPhenomenon commented Mar 8, 2021

Closes #1838
Related to #1861

@yulgolem
Copy link
Contributor

yulgolem commented Mar 8, 2021

@OlegPhenomenon BTW could you, please, take a look at #1863 regarding to this PR? I suppose the topic is pretty close.

@OlegPhenomenon OlegPhenomenon requested a review from yulgolem March 8, 2021 11:52
@yulgolem yulgolem self-assigned this Mar 16, 2021
@yulgolem yulgolem force-pushed the 1838-start-force-delete-process-for-email-addresses-in-bounces-emails-list branch 2 times, most recently from 929e0f5 to 97c295a Compare March 17, 2021 09:29
@yulgolem yulgolem changed the title test: added test for force delete bounced mail address Auto ForceDelete on email bounces/verification fails Mar 17, 2021
@yulgolem yulgolem force-pushed the 1838-start-force-delete-process-for-email-addresses-in-bounces-emails-list branch from 9344efc to c2d6158 Compare March 17, 2021 10:15
@yulgolem yulgolem assigned OlegPhenomenon and unassigned yulgolem Mar 18, 2021
@yulgolem
Copy link
Contributor

Added a rake-task to create dummy BouncedMailAddress:

bundle exec rake email_bounce:create_test[<some_email_address>]

@yulgolem yulgolem force-pushed the 1838-start-force-delete-process-for-email-addresses-in-bounces-emails-list branch from 1794b8b to 2bde087 Compare March 19, 2021 11:59
@yulgolem
Copy link
Contributor

Need to add registrant contacts as well.

@yulgolem yulgolem force-pushed the 1838-start-force-delete-process-for-email-addresses-in-bounces-emails-list branch 2 times, most recently from c17cb8b to 12ac617 Compare March 19, 2021 15:06
@vohmar
Copy link
Contributor

vohmar commented Mar 19, 2021

we need to add info about the reason for force delete to poll messages - ie "Force delete set on domain autofd1.ee. - invalid email: [email protected]"

@vohmar vohmar assigned yulgolem and unassigned vohmar Mar 19, 2021
@yulgolem yulgolem force-pushed the 1838-start-force-delete-process-for-email-addresses-in-bounces-emails-list branch from 6324374 to f7655b6 Compare March 22, 2021 09:26
@yulgolem yulgolem force-pushed the 1838-start-force-delete-process-for-email-addresses-in-bounces-emails-list branch 2 times, most recently from bab7381 to aa34959 Compare March 22, 2021 09:51
@yulgolem yulgolem force-pushed the 1838-start-force-delete-process-for-email-addresses-in-bounces-emails-list branch from aa34959 to 91093b2 Compare March 22, 2021 09:54
@yulgolem yulgolem force-pushed the 1838-start-force-delete-process-for-email-addresses-in-bounces-emails-list branch from 62d7c5a to 29f7aa1 Compare March 22, 2021 10:28
@yulgolem yulgolem assigned vohmar and unassigned yulgolem Mar 22, 2021
@vohmar vohmar merged commit 9fae797 into master Mar 22, 2021
@vohmar vohmar deleted the 1838-start-force-delete-process-for-email-addresses-in-bounces-emails-list branch June 8, 2021 13:36
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

Successfully merging this pull request may close these issues.

start soft force delete process for email addresses in bounces emails list
3 participants