I moved the task to the "Active" for further discussion about what to do here.
@longwave,
Thanks for the review. In this case, does it make sense to postpone this task for now and add to-dos or comments for the functions to add typehints later on? During some major changes later.
Because, for now, it looks like it's hard to continue with that task.
Seems like a problem was in the code. After the coding standards fixes in this task 📌 PHPCS compatibility Fixed condition was changed and it disables the checkbox if Content translation is enabled, but it should work in the opposite way.
I created a PR and added a patch to the change.
_shy → created an issue.
As I checked, it works with the Layout Builder blocks as well.
\Drupal\linkchecker\LinkExtractorService
service was able to extract links from the blocks, it happened on the Node layout update, here - linkchecker_entity_update()
Moved task back to RTBC according to the comment #29 ✨ Login/Logout event dispatcher Needs work
_shy → changed the visibility of the branch 3343518-add-deprecation-to to hidden.
Seems like this task is not relevant anymore as soon we have discussed the usage of the Hooks in other issues.
The new approaches discussed here:
-
🌱
[META] Hooks via attributes on service methods (hux style)
Active
-
📌
OOP hooks using event dispatcher
Needs review
I think it makes sense to close the current task.
On the project, we had a problem with unpublished blocks and paragraphs after the translation node. Added small changes to prevent unpublish blocks and paragraphs.
Seems like this issue is related to the admin theme. I suppose in the example, it was used as a Gin theme. Maybe some styles from there cause the bug.
Also, created a MR with changes.
I'm also getting the same error just in another file, here - \Drupal\office_hours\Plugin\Field\FieldType\OfficeHoursItemList
I prepared a quick patch for that fix.
Merged changes to the 1.x branch. Thanks.
I just updated the patch from comment #6 🐛 Layout builder fails after cloned page delete Needs work to be available for the last module version.
_motti,
Thanks, merged to the 1.0.x!
Merged to the 2.0.x-dev.
Thanks
Looks nice for me.
Merged to the 8.x-1.x branch.
Thank you, guys.
Merged to the 1.0.x.
Merged to the 3.x-dev!
I'll be happy to help with releasing Drupal 10 and higher compatible versions and help maintain this module. Also, I can opt for projects into security advisory coverage.
Please, add me as a maintainer.
Merged to the 8.x-1.x-dev.
Merged to the 8.x-1.x-dev.
Merged to the 8.x-1.x-dev.
Merged to the 8.x-1.x-dev.
Merged to the 2.0.x-dev!
Merged to the 3.x-dev!
Merged to the 3.x-dev branch. Thanks.
Merged to the 8.x-1.x-dev, thanks!
_shy → created an issue. See original summary → .
Module doesn't support Drupal 11 now.
Make the module compatible with D11
_shy → created an issue.
Merged to the 1.0.x.
Thanks!
Seems like it's not possible to access some links from the mega menu. I tried to open a "Drupal CMS" link from the menu and instead of redirecting just another mega menu item was opened.
Here is an example:
Merged, thanks!
These changes will be added to the next release.
Looks great, thanks.
Merged to the 1.0.x-dev and will be added to the next release.
vieilours, thanks for your notice.
I created an MR with fixes for the given issue. Added a validation to prevent an error for the entities without the Edit form template.
Now, we need some testing for it.
_shy → made their first commit to this issue’s fork.
Hi, it's a great idea. I reviewed and merged a PR. Will add these changes to the next release.
Thanks for your contribution.