Tom Rees-Herdman
Tom Rees-Herdman
Hey @kuasha420, nice work on the IB here, and good find for that ` should-run-github-action` action! I've got a few fairly minor comments to make, please see what you think....
Thanks @kuasha420! This is looking good to me; I have made a couple of minor tweaks which you can see in the edit history. One question, regarding the event types...
Hey @kuasha420, thanks for clarifying that. Somehow I failed to register that point in the IB and make the connection. Well, that being the case, this looks good to go!...
Have moved to MR for an additional merge review from @felixarntz and @marrrmarrr as per the AC.
Having discovered that the VRT suite is not actually running as expected in CI, I am parking this issue until that has been resolved: https://github.com/google/site-kit-wp/issues/6151
Based on the work on #6151, it looks like this issue will still be relevant, with a reduction in the number of test failures but still a couple of cases...
I think it could be worth keeping to see about fixing the timeouts that occur when running locally...
Hi, I am also seeing the same error for `md-menu-item`: 
As this was previously in Approval and the only change in the followup was to update the VRT images, and with CI confirmed passing for the update, I'm moving this...
Hi @zutigrm, the IB is looking good, however I am not sure we should use the `notification` query param as the condition for reshowing the banner and continuing to the...