- Issue created by @merilainen
- 🇮🇳India mohd sahzad
Mohd Sahzad → made their first commit to this issue’s fork.
- 🇫🇮Finland merilainen
Here is a patch to fix the situation. This will also upgrade the openai-php/client version to ^0.8.4.
- Status changed to Needs review
9 months ago 12:24pm 12 February 2024 - 🇫🇮Finland merilainen
I don't understand the content of the merge request, it doesn't change the library version but instead adds a "self-dependency" to the module itself in composer.json?
https://git.drupalcode.org/project/openai_image_for_drupal/-/merge_reque... - 🇿🇦South Africa rawdreeg
Not sure what issue this is describing. Are you sure you don't have a local version of openai-php/client locked to v0.8.4? Maybe check your project composer to address the dependency clash
- Status changed to Needs work
7 months ago 3:46pm 4 April 2024 - 🇿🇦South Africa rawdreeg
Not sure what issue this is describing. Are you sure you don't have a local version of openai-php/client locked to v0.8.4? Maybe check your project composer to address the dependency clash
- Status changed to Needs review
7 months ago 6:40am 8 April 2024 - 🇫🇮Finland merilainen
Yes the v0.8.4 requirement comes from another project and it will probably not work with the older v0.2.1. The problem is actually in the use composer caret ^ symbol, because:
"For pre-1.0 versions it also acts with safety in mind and treats ^0.3 as >=0.3.0 <0.4.0 and ^0.0.3 as >=0.0.3 <0.0.4"
https://getcomposer.org/doc/articles/versions.md#caret-version-range-
So I suppose it would be better to use the tilde ~ instead, so ~v0.2?
"Another way of looking at it is that using ~ specifies a minimum version, but allows the last digit specified to go up."
https://getcomposer.org/doc/articles/versions.md#tilde-version-range-
Here is a new patch with that suggestion.