Processing of activity id returned from sendEmail will fail in 5.36
Hi, In 5.36 as a result of dev/core#1750 (closed) to record separate activities for each email in order to record the replaced token values per-contact, when you process the activityId returned from CRM_Activity_BAO_Activity::sendEmail it's only acting on the last activity id if there were multiple recipients. The fact that some extensions do this type of processing was an oversight in the change, and there's a proposal at https://github.com/civicrm/civicrm-core/pull/19873 but it means one of two things for this extension:
If that proposal were put into 5.36, then without updating this extension and people deploying the updates it will silently fail in 5.36 (it will actually go and update activity id 1 !!). So that doesn't give much time.
But if that proposal were to go into 5.37, as currently targeted, then in 5.36 this extension will still silently fail, but in the sense of only processing the last activity id, and there's no way to really fix it without a temporary core hack until 5.37.
Wondering if you have any thoughts on the timing, or any better ideas. The latter seems the lesser of two evils.