The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
What's left here? Getting it passing?
- last update
over 1 year ago 29,448 pass, 2 fail - 🇭🇰Hong Kong hswong3i
Update MR https://git.drupalcode.org/project/drupal/-/merge_requests/1521 with 10.1.x
- 🇭🇰Hong Kong hswong3i
Re-roll https://git.drupalcode.org/project/drupal/-/commit/a5117b293edc1211c0e72... via 9.5.x-dev
- 🇦🇪United Arab Emirates ThirstySix
#102 Patch works well with D9.5.x version.
- Status changed to Needs review
3 months ago 6:43pm 20 September 2024 - 🇺🇸United States johne
I've updated existing tests for 10.x and all unit tests now pass for me locally. Unfortunately, I'm not able to apply this to 11.x and create a MR at this point.
- 🇺🇸United States smustgrave
Changing to 11.x as the current development branch
Was previously tagged for issue summary update not sue if that's happened
Fixes should be placed in MRs vs patches
- First commit to issue fork.
- Merge request !9608Issue #2429321 by prabha1997, aleevas, Rade, Neslee Canil Pinto, kalpaitch,... → (Open) created by herved
- 🇧🇪Belgium herved
Opened a new branch for 11.x based on the previous one.
Fixed tests based on #106. However I didn't include the ones that were added in #106 as there is a lot of code duplication and they are not passing.
Attaching also a static patch for composer.
