- Issue created by @acbramley
- last update
about 1 year ago 3 pass - @acbramley opened merge request.
- Status changed to Needs review
about 1 year ago 10:22pm 30 October 2023 - last update
about 1 year ago Composer require failure - 🇦🇺Australia elc
This may not be as simple as allowing the extra version - csv_serialization is 3.x branch for D9, and 4.x branch for D10.
Is there a way for this to be specified in composer.json format, or should csv_serialization also be split into two branches?
The require effectively needs to be: (drupal/core:^9, drupal/csv_serialization:^3) || (drupal/core:^10, drupal/csv_serialization:^4) which I don't believe is possible at present.
Another possibility would be hook_requirements informing of the correct version to install.
- 🇦🇺Australia acbramley
@ELC composer dependency resolution solves that for us. It won't update to 4.x unless the site is also on D10.
- 🇲🇺Mauritius j.b
Hello,
Can't we have a patch to correct the version in the composer.json while waiting for the merge request to be approved ? - last update
about 1 year ago 3 pass - 🇲🇺Mauritius j.b
just add this to your project composer.json
"drupal/csv_serialization": "4.0 as 3.0", "drupal/views_data_export": "^1.3",
then composer update
- Status changed to RTBC
about 1 year ago 12:39pm 31 October 2023 - 🇺🇸United States brooke_heaton
The tip in #9 helped me get this working in composer.json. Thanks @j.b.
A patch won't work if applied via Composer, since the composer.json is read prior to patching.
Can we get this committed, please?
- First commit to issue fork.
- last update
about 1 year ago 3 pass -
jhedstrom →
committed 46392e36 on 8.x-1.x authored by
acbramley →
Issue #3397869 by acbramley, j.b, solideogloria, ELC, nagender16,...
-
jhedstrom →
committed 46392e36 on 8.x-1.x authored by
acbramley →
- Status changed to Fixed
about 1 year ago 6:44pm 1 November 2023 - 🇦🇺Australia acbramley
Thanks @jhedstrom! Comparing 8.x-1.x to 8.x-1.3, this is the only change so a release would be amazing if possible!
Automatically closed - issue fixed for 2 weeks with no activity.