🇺🇦Ukraine @Tyapchyc

Made coreVersion method static

@amitaibu

I changed the field context name.

Tests added in tests/src/Kernel/MessageTest.php

Hello @liam-morland, @alan-d, @heddn, @nkoporec, @rconstantine, @webflo

Could you please review this?

Thank you

Hello @liam-morland, @alan-d, @heddn, @nkoporec, @rconstantine, @webflo

Could you please review this?

Thank you

Hello @liam-morland, @alan-d, @heddn, @nkoporec, @rconstantine, @webflo

Could you please review this?

Thank you

Hello @marcoliver, @peter-majmesku

Could you please review this?

Thank you

Hello @rob-holmes, @jhedstrom

Could you please check this?

Thank you

Hello @rob-holmes, @jhedstrom

Could you please review this?

Thank you

Hello @rob-holmes, @jhedstrom

Could you please review this?

Thank you

Hello @rob-holmes, @jhedstrom

Could you please review this?

Thank you

Hello @mandclu,

Could you please review this?

Thank you

Hello @mandclu,

Could you please review this?

Thank you

Hello @mandclu,

Could you please review this?

Thank you

Hello @mandclu,

Could you please review this?

Thank you

Try to reproduce with the latest module and Drupal version and enable the token module.
Work as expected.

@mandclu
yes, ready for review
Thanks

I added settings options to flag inputs as required.

Tests fixed 📌 Drupal 10.2/10.3 test fixes Needs review

tyapchyc → created an issue.

I'm not sure about what needs to be implemented. I added the possibility to set a timezone from twig (smart_date_format('extended', 'UTC')). But if we leave it empty they use core DateFormatter::format and set the default timezone.

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

https://www.drupal.org/project/message_ui → This module already implements access to messages per bundle by permissions.

To manage permissions to view messages you can use an additional module https://www.drupal.org/project/message_ui →
But in your case, you are using the additional field "field_message_to" to check access, so you need this custom code.

It looks like after updating the module you ran updb and configs was updates ( plugin value from "message_delete_action" to "entity:delete_action:message"). And then you needed to export and commit the changes of the config files, because if not, you'll try to import all configs to overwrite all the values ​​of a plugin that no longer exists

Sending emails to site email can be disabled in notification settings form ("Disable the site email address")

Sending emails to ADHOC Mail works.
I tried to reproduce this issue on drupal 10.2.7 /
8.x-3.x-dev (Mailpit to catch emails), can not reproduce, need either more details or close this issue.

Added information about access to view the entity to fields on notification form.

Added the ability to configure the user reference field from the entity to be moderated in the notification.
You need to add the user reference field to the moderated entity and configure it on the notification. And when a user will create new content, they can choose a user/users to moderate.

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

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

Was fixed in related task Notification emails should not send to blocked users →

resolved in https://www.drupal.org/project/content_moderation_notifications/issues/3... 📌 Do not send notifications when an entity is syncing (e.g. during a migration) Fixed

I tested and cannot reproduce this problem on last stable and dev versions.
Could you test this issue on last version of module (8.x-3.6) and close the issue or add more information.

Thanks.

If selected "Email the author?" option or/and unchecked "Disable the site email address" I received email notification about transition.
So I can not reproduce current issue on 8.x-3.5 version or later.
Could you please add more information or close the issue.

Thanks

I tried to reproduce this issue on drupal 10.2.7 /
8.x-3.x-dev (Mailpit to catch emails), can not reproduce, need either more details or close this issue.

Added access check to query.

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

Fixed default value for single options.

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

Added access check to query loads nodes of a selected content type.

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

+