Closed aaemnnosttv closed 9 months ago
I went through our usual QA checks:
Note: there are several UI issues related to 7968 and 7542. Mohit and I have documented them on Slack, so when the two tickets are merged, we can do a full check of all icons/SVGs.
Some observations:
Aside from alignment issues that are being fixed in #7968, all looks good to me. 👍🏻
:white_check_mark: Performed a general smoke test of the plugin. :warning: Verified/smoke tested the changes I’ve been involved with. :white_check_mark: Performed some ad-hoc testing.
While testing #6584 I noticed a minor glitch in the loading state for the AdSense setup screen. The screen initially displays with a progress bar, but then briefly shows the spinner button with the spinner active. All of the loading should have taken place by the time the button is shown.
https://github.com/google/site-kit-wp/assets/18395600/fda6e88a-b0b3-4ed1-8ed9-1f0f6d911c4c
This is a minor defect that we could potentially fix in a followup issue.
LGTM 👍
LGTM too!
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance criteria
Implementation Brief
QA Brief
Changelog entry