@joaopauloscho

breidert → credited joaopauloscho → .

Nice, it makes sense! Tested it and noticed an error related to the system_requirements() function, so I've pushed a fix for it. Can you review please? Thanks.

Please review the MR, thanks.

I'm working on this.

Reviewed and tested the patch and created the MR. Please review it. Thanks!

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

Fixed the conflict but the 1.x branch requires the ckeditor contrib module and it doesn't seem to have Drupal 11 compatibility, so the pipeline has failed. I'm amending this to Needs Work.

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

I'm not sure if https://www.drupal.org/project/hubspot/issues/3388150 🐛 Fatal error trying to add HubSpot handler in webform RTBC would be a duplicate of this but I'm adding psr/http-message:^1.1 as a requirement along with hubspot/hubspot-php:^5.0 . The reason of this is because the hubspot newest version (5.2) requires psr/http-message::^1.1 now. Please take a look, thanks!

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

Patch #5 applied fine and fixed the warnings for me.

Please review the updated patch for the 6.0.x.

Thanks.

xurizaemon → credited joaopauloscho → .

Updated the patch from #44 to work with 10.2.2

I'm having the same issue on 2.0.0-alpha0 and also agree that array_diff_assoc would be a better approach. Please review the patch.

Thanks!

Updated patch to fix deprecated function log.

The solution from #32 worked for me, thanks.

Updated the patch to work with the module's new version.

I got this error when accessing the sitemap of a d10 multi-domain site. Applying the patch works. I'm just adding a new one to update the @var type in the comment.

Changing to the jquery_ui project.

The patch works fine for me.

Updated patch to apply with version 2.

Updated the patch to apply with the latest version.

Patch from #12 also works for us, thanks.

Just updating de indexes to apply it smoothly.

RTBC +1

The MR works for us, thanks!

Patch #2 works for me too, moving to RTBC

Applied and it's looking good to me.

I was thinking, shouldn't we create a release to still require ckeditor 4?

Changed it to ^10, please review.

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

Adding a new patch with the suggestions.

#80 works for us, thanks.

#28 works for us, thanks! I'm just updating to apply it with the 9.5.9 version.

Adding return type to functions in order to fix errors.

The patch #11 works for us on 9.5.5 - Thanks!

The patch #159 works for us on 9.5.2 - thanks!