- πΊπΈ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.
Can the MR be updated for 10.1 please
Left a comment, but since it's updating the schema believe it will need an update hook and thus test for update hook. - Status changed to Needs review
10 months ago 6:49am 16 August 2023 - last update
10 months ago 29,465 pass, 2 fail - last update
10 months ago 29,466 pass - πΊπ¦Ukraine nginex
I cound not create a new MR against 10.1.x as I'm not owner of the existing MR.
But I rerolled existing MR against 10.1.x
The last submitted patch, 43: 3180702-d10-43-TEST-ONLY-FAIL.patch, failed testing. View results β
- Open on Drupal.org βEnvironment: PHP 8.1 & MySQL 5.7last update
10 months ago Not currently mergeable. - last update
10 months ago 29,962 pass - πΊπΈUnited States tim.plunkett Philadelphia
@nginex I rebased the MR (which anyone can do) and @joachim changed the target (which I asked for in Drupal slack)
- πΊπ¦Ukraine nginex
@tim.plunkett, right, I will take this into account next time, thanks
- Status changed to Needs work
10 months ago 2:36pm 16 August 2023 - πΊπΈUnited States smustgrave
Previously tagged for update path for the new schema item.
If not needed can we document why not please.
Looking good though!
Thanks!
- Issue was unassigned.
- πΊπ¦Ukraine alt.dev
I'm attaching a patch based on the latest MR that can be applied to Drupal 10.2.x.
- πΉπ·Turkey burak.ikiler
Current d10-49 patch is failing with 10.2.3. I am adding the compatible one.