Tom Rees-Herdman

Results 525 comments of Tom Rees-Herdman

Thanks for catching that @nfmohit! You are spot on, it should be the other way round. I've fixed the IB, back over to you! 🦅 👁

Hey @jimmymadon, it's worth noting we have a similar issue in the backlog, https://github.com/google/site-kit-wp/issues/5561, where we looked into hyphenation, among other things - see https://github.com/google/site-kit-wp/issues/5561#issuecomment-1191770816. However as discussed there are...

Thanks for flagging this @wpdarren, it turns out the PR was approved but not merged although the rest of the process was completed with the issue moved to QA. This...

Moved to Backlog until we get all the P1 issues done and/or the initial release out.

Note that as discussed in #9983, we are probably going to remove the Connect Analytics CTA from the codebase. If so, we can close this issue.

Thanks @ankitrox! We can indeed close this issue.

Thanks @benbowler, interesting to hear the progress to date and it's a good point that updating the various package versions across the board is the preferred long time solution, it's...

> Thanks @techanvil, I've updated and tested the node version to `node:16.14.2-bullseye` and confirm this still appears stable. > > I've written up the current state to an IB to...

Thanks @nfmohit, however, I don't think we should actually make that a requirement here. The favoured approach for the server side sync is to use the WP Cron scheduling system,...

Thanks @nfmohit. Bumping the estimate up sounds reasonable to me given the scope of the implementation. @ankitrox, a small detail, to meet the AC where _the task should only be...