Darren Cronian

Results 198 comments of Darren Cronian

### QA Update: ✅ **Verified:** - The `view_notification` GA event for the activation category is triggered only when the ActivationApp notification is displayed. - The `view_notification` GA event for the...

### QA update: ✅ **Verified:** - Checked out the entire Site Kit workflow and verify that there are no regressions. - Set up Site Kit on a new site with...

@benbowler There are two issues with this notification. 1) The AdSense title, i.e logo, and text are small. 2) the SVG you highlighted above must be smaller and more aligned...

@benbowler These notifications appear from the AdSense API and are a little different than normal notifications, so I am not sure we have anything to compare with. My thoughts are...

### QA Update: ✅ **Verified:** - On the Settings page, the Visitor groups section with the toggle switch is displayed under the Key Metrics section. - The toggle switch is...

### QA Update: ✅ **Verified:** - The tag is output as expected without any changes, except for adding the comment. ![image](https://github.com/google/site-kit-wp/assets/73545194/4df08057-3620-4e75-a899-9f538e9e6eaf) ![image](https://github.com/google/site-kit-wp/assets/73545194/645413cd-7d8f-435c-83b8-7f53eb8e15a8)

### QA Update: ✅ **Verified:** - Completed a smoke test of every module and feature within Site Kit. - Focused on the behavioural aspects, in particular the UI. - Site...

> It's not something to block the issue or release over so we can look into addressing this going forward @aaemnnosttv is it worth creating a ticket for this?

### QA Update: ✅ **Verified:** - Opened SK on mobile with a width

### QA update: ⚠️ @eugene-manuilov @techanvil tagging you in the absence of Arafat. Within the tester plugin I have set AdSense to `Approved` and `Added`. The `adsenseSetupV2` is not enabled....