alexpott → credited kjankowski → .
The bokbasen/php-api-client change has been accepted, so this MR is safe to merge now.
Added changes on the fork that adds a new GuzzleHttps/Client with the configuration needed.
This depends on the client lib: bokbasen/php-api-client change of the argument type (MR) being merged, otherwise it won't work.
kjankowski → created an issue.
Added handling of the missing keys in the cache array.
There was a similar issue with support for guzzle 7 on geocoder (
https://www.drupal.org/project/geocoder/issues/3283651
✨
Drupal 10 compatibility: changing php-http/guzzle6-adapter dependency into php-http/guzzle7-adapter
RTBC
)
It's difficult to keep the full compatibility with Drupal 9 & 10 - even though guzzle 7 is also allowed on D9 now (
https://www.drupal.org/project/drupal/issues/3225966 →
), drupal/core-recommended is locked on guzzle 6, so it may cause some dependency problems.
As suggested in the description, adding a new 5.x branch seems like the best way to go.
I fixed all tests for PHPCS & PHPStan - seems ready for D10 now, can you review?
kjankowski → made their first commit to this issue’s fork.
kjankowski → made their first commit to this issue’s fork.
Added a small fix that will prevent showing warnings in the editor whenever a new image style is added.