- Issue created by @eiriksm
- ๐ฎ๐ณIndia viren18febs
Hi @esolitos
I have added patch file for above MR, please review.
- Status changed to Needs review
over 1 year ago 10:11am 8 November 2023 - ๐ต๐ฑPoland kjankowski Warsaw
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. - First commit to issue fork.
I just fixed as much as I could. Fixed the tests and a lot of codestyle errors.
- Status changed to Needs work
about 1 year ago 2:45pm 4 December 2023 Not ready yet. The composer.json requires php-http/guzzle7-adapter (for compatibility with D10 core requirements), but zaporylie/vipps:^2 requires php-http/guzzle6-adapter
- ๐ณ๐ดNorway zaporylie
zaporylie/vipps doesn't require php-http/guzzle6-adapter
- Status changed to Needs review
12 months ago 11:03am 28 February 2024 - ๐บ๐ฆUkraine Oleksandr Yushchenko
oleksandr yushchenko โ made their first commit to this issueโs fork.
- ๐ณ๐ดNorway eiriksm Norway
I can not approve the MR in gitlab, but I reviewed it nevertheless :)
-
oleksandr yushchenko โ
committed 770cb242 on 8.x-4.x authored by
esolitos โ
Resolve #3388269 "Drupal 10 compatibility"
-
oleksandr yushchenko โ
committed 770cb242 on 8.x-4.x authored by
esolitos โ
- ๐ณ๐ดNorway esolitos Trondheim
oleksandr yushchenko โ credited esolitos โ .
Automatically closed - issue fixed for 2 weeks with no activity.