Skip to content

Conversation

@alansaun92
Copy link
Contributor

Running upgrade status highlighted that this module was not compatible with Drupal 11. This pr adds in Drupal 11 support and drops Drupal 9.

@millnut
Copy link
Member

millnut commented Aug 30, 2024

Thanks for this @alansaun92, I don't think we can merge this just yet as we have to check all the dependencies and contrib dependencies of this module to see if we need to create a new major version. I'm currently working through a list that should give us better visibility of the dependencies.

@alansaun92
Copy link
Contributor Author

That's understandable and I agree @millnut. I know that fixing all of the issues that upgrade status "says" exists isn't always correct. Trying to get all of the dependencies aligned is the fun part.

@ekes
Copy link
Member

ekes commented Sep 3, 2024

@ekes ekes marked this pull request as draft September 3, 2024 08:25
@millnut millnut requested a review from ekes January 23, 2025 11:35
@millnut millnut marked this pull request as ready for review January 23, 2025 11:35
@millnut
Copy link
Member

millnut commented Feb 4, 2025

@ekes @stephen-cox this looks ok to merge, all green

@finnlewis finnlewis merged commit 3824d81 into 3.x Mar 4, 2025
17 checks passed
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.

6 participants