Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ops 10638 weekly updates #858

Closed
wants to merge 8 commits into from
Closed

Conversation

lazysoundsystem
Copy link
Contributor

Refs: OPS-10638

This moves to Drupal 10.3.

https://www.drupal.org/project/drupal/issues/3047110 has been merged, so the patch needed removing.

https://www.drupal.org/project/drupal/issues/3008292 is problematic - there's a new version of the patch on the issue, which didn't apply - the previous comment concludes the patch is no longer necessary. I haven't tested this yet.

https://www.drupal.org/project/imageapi_optimize_webp/issues/3453941 is needed to allow the site install. This patch has been added to all the other repos that are on 10.3.

Plus stage_file_proxy:^2 is no longer supported.

@lazysoundsystem
Copy link
Contributor Author

This has got messy - need to look more closely at the changelog. Very blocked.

@lazysoundsystem lazysoundsystem added the blocked This PR is blocked, do not merge it label Jul 25, 2024
@lazysoundsystem
Copy link
Contributor Author

@orakili - I was hoping to summarize this more cleanly before you get to updating core to 10.3, but some of this might be useful.

With these changes the automatic updates with config job runs (i.e. the site install part works which was blocked by the html/modules/custom/reliefweb_files/src/Controller/ImageStyleDownloadController.php change), but the tests are still failing and I haven't yet worked out why.

@orakili
Copy link
Collaborator

orakili commented Aug 5, 2024

Replaced by #865.

@orakili orakili closed this Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This PR is blocked, do not merge it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants