Problem/Motivation

Drupal 11 compatibility

Issue fork webform_email_reply-3546408

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

intrafusion created an issue. See original summary.

intrafusion opened merge request !9

intrafusion’s picture

intrafusion
Primary language English
Location Edinburgh, UK
commented
Status: Active » Needs review
vinodhini.e’s picture

vinodhini.e commented

Hi @intrafusion,
Installed the module on Drupal 11.2.3.
Encountered compatibility issues with Drupal 11.
Applied the provided Merge #9 to fix the issues.
Verified that the module is now compatible with Drupal 11. Thanks.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Status: Needs review » Reviewed & tested by the community

Sounds like @vinodhini.e has tested it in #4, setting the status to RTBC

martijn de wit made their first commit to this issue’s fork.

martijn de wit’s picture

martijn de wit
Primary language Dutch
Location 🇳🇱 The Netherlands
commented

removed ^8 || ^9 compatibility

martijn de wit’s picture

martijn de wit
Primary language Dutch
Location 🇳🇱 The Netherlands
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

The pipeline looks cancelled... strange but this commit showed up on in a new MR
https://git.drupalcode.org/project/webform_email_reply/-/merge_requests/... from this issue #3566529: HTML email body is escaped when using CKEditor reply message

fjgarlin’s picture

fjgarlin commented

The pipeline was cancelled manually, so it is expected.

Re the commit showing up on another MR, it's just due to the target of that MR being wrong. I commented on it.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

Thanks for fixing this so quickly @martijndewit and helping @fjgarlin!

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Version: 2.0.x-dev » 3.0.x-dev

I'll change the version here too while I am here on the issue.