- 🇺🇸United States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
If we are fixing a view from core shouldn't this need an upgrade path?
- Status changed to Needs review
over 1 year ago 11:14am 7 July 2023 - last update
over 1 year ago 29,803 pass - 🇮🇳India ranjith_kumar_k_u Kerala
Added upgrade path and test, please review
- last update
over 1 year ago 29,433 pass, 2 fail - Status changed to Needs work
over 1 year ago 3:05pm 7 July 2023 - 🇺🇸United States smustgrave
The issue summary could use some love
Reviewing the patch though
$this->assertNotEquals($empty_text_c
1. shouldn't this be empty?node_update_10100
2. Needs to be updated for 10.2 - Status changed to Closed: outdated
over 1 year ago 3:18pm 7 July 2023 - 🇺🇸United States smustgrave
So actually this appears to be outdated according to https://developer.chrome.com/docs/lighthouse/best-practices/external-anc...
But will move over credit to 📌 [Meta] Bug Smash Initiative triage fortnight commencing 2023-07-04 Fixed