🇺🇸United States @matthand

Hi @cmlara,

The assumption that the scheme will always be public is based on the routes defined in the Image module. See https://api.drupal.org/api/drupal/core%21modules%21image%21src%21Routing... where the directory path is set by using the method `->getViaScheme('public')`.

It looks to me like this MR fixes the bug introduced by the update to Drupal 10.3. If we need to make other improvements isn't that better to do in the issue you linked?

matthand → created an issue.

I still see this as an issue in Drupal 10. Even clearing the browser cache data doesn't easily solve the problem for the visitor. If a visitor already visited the website previously, the cached version of the logo and favicon is displayed even if there is a newer version of the files. Adding a cache busting query string looks like a good solution to me.

Is there a way to re-enable the UUID field that is disabled on node resources by JSON:API without this new enableField method? I don't think so. Real world scenario of trying to match a legacy Drupal 8 JSON:API endpoint to prevent consumer apps requiring a rebuild.

geocalleo → credited matthand → .

matthand → created an issue.

Merged!

matthand → created an issue.

matthand → created an issue.

matthand → created an issue.

matthand → created an issue.

matthand → created an issue.

This is reroll of the patch file from comment 71 against the latest release 3.1.

Hi jonathan1055, We rely on the Scheduler module for our Drupal 9 websites. We want to clear our path to Drupal 10 upgrade. I can see that Scheduler only has one more bug fix pending and some docs changes before a stable release. What help do you need? We will start by testing out the current patch for https://www.drupal.org/project/scheduler/issues/3336108 🐛 Don't run hook_modules_installed during config install Fixed .

We are interested in seeing this fix released for Scheduler module. Is it possible to make a new release candidate? What do you need help with?

Starting progress on the Twig filter.

I merged a drupalci.yml file into 2.x branch. This should give us a good starting point for CI integration. I could use a review to ensure I'm using reasonable defaults for my project.

matthand → created an issue.

matthand → created an issue.

matthand → created an issue.

matthand → created an issue.

matthand → created an issue.

Since there's little usage of the Drupal 7 release, there's no need to add testing support for that version. We added Kernel tests to the 2.x branch and will release soon when the stable version 2.0 is issued.

Tests are now available for the 2.x branch.

In progress adding tests to 2.x-dev.

Changing to new default branch of 2.x-dev.

Hi @pub497,

Thank you for submitting the bug report. I'm not able to reproduce your bug. Can you please check that you ran 'drush cr' before the 'drush fr' commands? We checked the latest release and there's an update hook that should run and flush the container cache. This would fix the error your seeing.

Features version 8.x-3.13 is released → that drops support for D8 and adds support for D10. Great work community! Thank you for your contributions.

Features version 8.x-3.13 was just released → and includes this new enhancement. Thank you for your contribution!

Features version 8.x-3.13 was just released → and includes this bugfix. Thank you for the contribution, Dave!

Features Version 8.x-3.13 was just released → and includes this fix. Thank you for your contribution.

Fix is now merged into 3.x-dev branch. Thank you, Dave!

There was a manual check for a "yes" argument that was not needed and causing the log of the PHP Warning message. This is now resolved using the MR branch in local testing with Drush.

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

Thanks for reviewing and testing community. This change looks ready to merge into 3.x-dev branch. Marking as fixed.

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

This fix is merged into 3.x-dev. We are going to commit a few more issues and then cut a new stable release for the 3.x branch.

Reviewed the code in GitLab. This merge request was already running on Drupal 10 websites. The D10 test is passing and D9 with 3 versions of PHP. Let's merge this in and cut a new stable release.

@nedjo, I work with @dave-reid and would be happy to volunteer for co-maintainer.

Confirming the patch in comment 9 applies clean from composer.json and resolves this issue. It also fixes the related issue
"Local Tasks menu is shown even when the tray is empty" 🐛 Local Tasks menu is shown even when the tray is empty Closed: outdated . Moving to status "Reviewed & tested by the community".