Skip to content

Conversation

@tonypaulbarker
Copy link
Contributor

Update patch to resolve #886

@millnut
Copy link
Member

millnut commented Sep 1, 2025

Hi @tonypaulbarker just checked the action log and looks like the newer patch doesn't apply, I think that ticket for redirect needs to be re-rolled against the latest version of redirect.

Gathering patches for dependencies. This might take a minute.
  - Installing drupal/redirect (1.12.0): Extracting archive
  - Upgrading localgovdrupal/localgov (3.x-dev 7ae8471 => dev-fix/3.x/886-drupalredirect-patch-update 9007e12): Extracting archive
  - Applying patches for drupal/redirect
    https://www.drupal.org/files/issues/2024-03-18/3431260.patch (Create redirect from path alias change and workspaces https://www.drupal.org/project/redirect/issues/3431260)
    https://www.drupal.org/files/issues/2025-08-07/redirect--2025-08-07--3057250-80.patch (Validation issue on adding url redirect: https://www.drupal.org/project/redirect/issues/3057250)
   Could not apply patch! Skipping. The error was: Cannot apply patch https://www.drupal.org/files/issues/2025-08-07/redirect--2025-08-07--3057250-80.patch

@tonypaulbarker
Copy link
Contributor Author

tonypaulbarker commented Sep 1, 2025

@millnut @finnlewis I looked at a patch for the contrib issue but there are many differences from the last one and confirmed on the issue no patch ready yet. Shall we pin the version for now?

@tonypaulbarker tonypaulbarker changed the title Update redirect patch and bump redirect to 1.12 Set version constraint for redirect >=1.10 <1.12 Sep 2, 2025
Copy link
Member

@finnlewis finnlewis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good to me, thanks @tonypaulbarker

Fixes the install and keeps things stable while the redirect issue / patches get tested and fixed.

@finnlewis finnlewis merged commit 6bee491 into 3.x Sep 2, 2025
12 of 13 checks passed
@finnlewis finnlewis deleted the fix/3.x/886-drupalredirect-patch-update branch September 2, 2025 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

drupal/redirect patch no longer applies

4 participants