Page MenuHomePhabricator

Suppression list logic - sanity check
Open, Needs TriagePublic4 Estimated Story Points

Description

I've always been a bit quesy about our suppression list logic & just want to sanity check it

The suppression list is

  • all emails in log_civicrm_email where
    1. the email is not in silverpop_export_staging OR
      1. the record is more recent than the latest in silverpop_export_staging

silverpop_export_staging holds all primary email addresses from non-deleted contacts

This leaves the situation where a contact has more than one email. My understanding is that the non-primary will wind up on the suppression list - which is OK until it becomes primary but I don't think it will come OFF the suppression list if it does.

I have a feeling this might have come up before & been determined to be 'OK' due to the low volume & the annual-ish purge of the MSL

Event Timeline

Dwisehaupt set the point value for this task to 4.Dec 19 2022, 9:11 PM

@AKanji-WMF I think we should think about this as part of any push on Acoustic or opt out logic - perhaps we could better use tags to group phabs by topic area? we do have some (dedupe, drupal) but....

@AKanji-WMF - should I review this with Ilse & Mariana before we do a ton rewiring the donor portal?

@SBurnett-WMF - adding in @Lars - this is about the master suppression list, right? and just a consideration as we think about how we are taking forward this work.

@SBurnett-WMF and @AKanji-WMF this is already an active discussion in the context of how contacts come off of the MSL.