[00:04:31] MaxSem: hrm, I'd prefer a "membership X will expire soon" hourly chron that warns beforehand (via echo events). For very short-term groups, a delayed job could be used. [00:05:33] AaronSchulz, basically that's what I'm doing in https://gerrit.wikimedia.org/r/#/c/350500/ [00:05:55] I can think of a billion ways tis could go wrong though [00:07:10] thought I guess if the message is "will expire in one hour", then you may as well just trigger it for 1 hour groups rather than even use a job. [00:07:15] so what's wrong about hooks in transactions [00:07:22] I'm still not sure notification are the best way to present this. [00:07:58] Ideally, it might be better to do it permission errors instead (but that requires some GUI cooperation among extensions). [00:08:01] it needs 2 types: forewarning (say, couple of weeks) and "poof, tis gone" [00:14:37] it would also probably be a good idea to have non-chron purges only purge stuff that has been expired for days. Delayed jobs would need to do a CAS style check on gu_expiry in case they are no longer relevant (due to manual removal 20 minutes before expiry or something). [01:19:56] bd808: my OCD got the better of me!