Tom Rees-Herdman
Tom Rees-Herdman
Hi @zutigrm, thanks for drafting this IB. First off, I would say I think it's a pretty ingenious idea, and I think could be a fitting solution in a different...
Hey @aaemnnosttv, thanks for the detail here. One thing though, although we can presumably still trigger `wp-cron.php` when `DISABLE_WP_CRON` is defined, it does seem that conceptually we're not "supposed" to,...
Thanks @aaemnnosttv, sounds reasonable. I guess this one's back over to you @zutigrm for a fresh look, taking Evan and my feedback above into account. I know there are still...
Thanks @zutigrm, the service you have outlined looks pretty good. That said, I have to confess that the further we get into this the more I'm wondering if we are...
Thanks @aaemnnosttv, good spot. I think we can implement that in #8264, but I've updated the POC to accommodate this in a slightly cheap and cheerful manner. Actually this makes...
I'm moving this back to AC as we need to determine whether or not it's actually needed for the Consent Mode MVP.
@bethanylang thanks for spotting this. With this issue removed from the CoMo epic, #8273 can also be removed. I have removed that issue from CoMo and assigned both of these...
Hey @abdelmalekkkkk, thanks for drafting this IB. One small detail - we don't want to set the `slug` query param in `KeyMetricsSetupApp` when `isInitialSetupFlow` is true, only the `notification` param....
Thanks @abdelmalekkkkk, that's great. LGTM! IB ✅
Hey @hussain-t, thanks for drafting this IB. One small detail - we shouldn't set `gaEventLabel` for the menu item, as we haven't yet defined the GA events for the updated...