This is a follow up to getting mailing retrieval going. There were 2 mailings retrieved by getMailings that 'didn't exist' - causing the script to fail. I worked around it by fetching mailings either side. Only 2 in 450 days but not sure if it will happen again.
eg. 52835553 - these caused the script to crash & I got around it by just skipping a time portion.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| Make mailing get process skip on exception. | wikimedia/fundraising/crm | master | +31 -25 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | None | T161757 EPIC: Add key silverpop data into Civi | |||
| Resolved | Eileenmcnaughton | T161758 retrieve the text/ html and statistics data for mail we have sent | |||
| Resolved | Eileenmcnaughton | T171433 Do we need to add handling for the 2 silverpop mails that would not retrieve properly |
Change 368940 had a related patch set uploaded (by Eileen; owner: Eileen):
[wikimedia/fundraising/crm@master] Make mailing get process skip on exception.
Change 368940 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Make mailing get process skip on exception.