Problem/Motivation

3.0.x branches info file only specifies drupal 8 and 9
openapi.info.yml
core_version_requirement: '^8 || ^9'

While composer.json supports 8, 9 and 10
core_version_requirement: ^8 || ^9 || ^10

This means one can use composer to download it, but then can not enable it on drupal 10.

Steps to reproduce

composer require 'drupal/openapi:3.0.x-dev@dev'
drush en openapi -y

In ModuleInstaller.php line 123:

Unable to install modules: module 'openapi' is incompatible with this version of Drupal core.

Proposed resolution

just update the composer.json, easy.

Remaining tasks

update composer.json

User interface changes

none

API changes

none

Data model changes

none

Issue fork openapi-3451937

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

thursday_bw created an issue. See original summary.

sepa_cleversoft’s picture

Is enough to change the yaml file to make the version 4.x of the module compatible with Drupal 10?

Do you know if it is working with json:API to serve openAPI version 3 version of the Drupal jsonapi resources? Or are still there some issue to make it fully compatible?

sepa_cleversoft’s picture

I've tried with Drupal 10, openapi 4 and openapi_jsonapi 3.

Everything was fine during composer requires and crush installations.

When I tried to open the page with the json output I got the white page with error.

Then something strage happened: composer, for example with composer up, started trying to apply 2 patches to the core, failing.

Then the website became inaccessible and drush was broken. My only chance was to restore a backup.

I'm not sure what was the event that fired the Armageddon, but it happened.

Do anybody have a clue on what happened? :D

ankitv18’s picture

I guess this should be covered into D11 compatibility.

bradjones1’s picture

This module is not yet stable. Thanks for pointing this out. Please do not use 3.x in production (yet).

bradjones1’s picture