- Issue created by @ignaciolflores
- @ignaciolflores opened merge request.
- Status changed to Needs review
over 1 year ago 12:10pm 30 March 2023 - 🇪🇸Spain ignaciolflores
Changes made don't affect tests on the project. I think that changes on MR28 can be reviewed.
- 🇭🇺Hungary mxr576 Hungary
Changes made don't affect tests on the project. I think that changes on MR28 can be reviewed.
Indeed, 8.x-3.x fails on D10 testing, I have started a D9 build (just like on my issue) and it passed.
- Status changed to RTBC
over 1 year ago 1:52pm 4 April 2023 - First commit to issue fork.
- last update
over 1 year ago 14 pass, 1 fail - last update
over 1 year ago 17 pass - Open on Drupal.org →Core: 10.1.x + Environment: PHP 8.1 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇵🇭Philippines angheloko
The patch in #1 doesn't work seem to work on D10, with Symfony 6.
I'm attaching a new one, tested on a fresh D10 install.
- Open on Drupal.org →Core: 10.1.x + Environment: PHP 8.1 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇵🇭Philippines angheloko
Patch in #8 will fail when `sort` is scalar. Here's a new patch that supports both.
- last update
over 1 year ago Composer require-dev failure - last update
over 1 year ago Composer require failure - last update
over 1 year ago Composer require-dev failure - Open on Drupal.org →Core: 10.1.x + Environment: PHP 8.1 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇺🇸United States JimSmith Oak Ridge, Tenn.
This bug was preventing Gatsby Cloud from pulling content to run a new build. I tested patch #9 and it resolved the problem. Thank you, @angheloko!
- last update
over 1 year ago Composer require-dev failure - 🇭🇺Hungary mxr576 Hungary
Hiden patch 8 because
Patch in #8 will fail when sort is scalar. Here's a new patch that supports both.
- Open on Drupal.org →Core: 10.1.x + Environment: PHP 8.1 & MySQL 5.7last update
over 1 year ago Waiting for branch to pass - 🇭🇺Hungary mxr576 Hungary
Based on
\Drupal\jsonapi\Controller\EntityResource::getJsonApiParams()
#9 contains the proper fix and MR28 should be closed or the$request->query->all('filter')
should be incorporated to the fix. - last update
over 1 year ago Patch Failed to Apply - last update
over 1 year ago Composer require failure - last update
over 1 year ago 17 pass - last update
over 1 year ago 17 pass - 🇳🇱Netherlands bbrala Netherlands
CI should be fixed again, so restarted a testjob.
- last update
over 1 year ago Patch Failed to Apply - last update
over 1 year ago Patch Failed to Apply - Status changed to Fixed
over 1 year ago 12:54pm 31 August 2023 - 🇳🇱Netherlands bbrala Netherlands
Thanks for the contribution. Great to have this working again.
Automatically closed - issue fixed for 2 weeks with no activity.