🇫🇷France @romainj

Patch working for me too.

I think that the media type labels are sorted alphabetically by default. But the sort is based on the english labels and so it is not sorted for other languages. I tested with the french language and yes the sort is based on the english labels. So if I switch to french the labels are not sorted as expected.
Patch #3 is not working for me.

romainj → created an issue.

romainj → made their first commit to this issue’s fork.

I have the same error with version 3.1.0:

LogicException: The controller result claims to be providing relevant cache metadata, but leaked metadata was detected. Please ensure you are not rendering content too early. Returned object class: Drupal\Core\Cache\CacheableJsonResponse. in Drupal\Core\EventSubscriber\EarlyRenderingControllerWrapperSubscriber->wrapControllerExecutionInRenderContext() (line 154 of /var/www/html/web/core/lib/Drupal/Core/EventSubscriber/EarlyRenderingControllerWrapperSubscriber.php).

This comes from the controller response:

opigno_learning_path/modules/opigno_user_selection/src/Controller/OpignoUserSelectionController.php

$response = new CacheableJsonResponse($response_data);
$response->addCacheableDependency($meta);

I'm not sure what to do here to return a proper cacheable json response.

Typo

Adds the Config Overview module to the list

romainj → created an issue.

romainj → made their first commit to this issue’s fork.