#LutskGCW25

The service declaration can be simplified as autowire is set to true globally for all services.

The service declaration can be simplified as autowire is set to true globally for all services.

The service declaration can be simplified as autowire is set to true globally for all services.

Having each hooks group in a separate issue may not be ideal as it will produce merge conflicts, but it looked too big for me to have a single MR.

Fully agree with the approach

The service declaration can be simplified as autowire is set to true globally for all services.

Thanks for the review!
I've pushed updates to the views hooks MR, but I'm unsure when I'll get to others.
Having each hooks group in a separate issue may not be ideal as it will produce merge conflicts, but it looked too big for me to have a single MR.
Probably, the rest of the issues should be put into postponed status and merged one by one after they are fixed.

The tests have failed, need to review and fix that.

I think it would be great to add a simple test that checks those links.

I did a quick review in the first from the list but I guess the remarks might be applied to all other child issues

The tests have passed, however, I'm not sure if we have any tests for the views integration so probably would be great to check if that works fine for Drupal 10.3+

We do have tests for D10, not sure about Views integration.

Great! I will take a look in the following days

Fixed, needs review and testing.

Closed because of git issue
Implemented in #3502242

Merged to the 8.x-1.x-dev.

Merged to the 8.x-1.x-dev.

I checked the code and tested the patch, and everything looks good.

I checked the code and tested the patch, and everything looks good.

I checked the code and tested the patch, and everything looks good.

I have added a checkbox. When it checks it will show a hidden textfield where we will add the URL for testing purposes.

snap_x → closed merge request !11

snap_x → closed merge request !8

snap_x → reopened merge request !8

Great, thank you for the fix!

snap_x → closed merge request !10

It works! Thank you!

Thanks!

Compatibility with Drupal 11 has been ensured.

I checked the code and tested the patch, and everything looks good.

Also updated comments into the functions in the UrlInspectionOperationsManager class

Merged to the 2.0.x-dev!

The Drupal 11 compatibility was added

Compatibility with Drupal 11 has been ensured.

I checked the code and tested the patch, and everything looks good.

Thanks for your MR

@aid_ua, I fixed the style code and now all the tests are successful.

faust74 → closed merge request !5

I rewrote a little bit of information from ReadMe.txt and added links into Readme.md

snap_x → closed merge request !9

LGTM.

snap_x → closed merge request !8

Thanks for the update @o_timoshchuk

Added general information how to set up the module

snap_x → closed merge request !6

Looks good for me, thanks!

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

The Drupal 11 compatibility was added

Module doesn't support Drupal 11 now.
Make the module compatible with D11

thanks for the MR

The tests have passed, however, I'm not sure if we have any tests for the views integration so probably would be great to check if that works fine for Drupal 10.3+