Skip to content

Comments

TP-1989 Changed ready to publish notifications to be sent for all organisation group admins.#1178

Merged
jussiviitamaki merged 1 commit intodevelopfrom
task/TP-1989_group-admin-notifications
Jan 15, 2026
Merged

TP-1989 Changed ready to publish notifications to be sent for all organisation group admins.#1178
jussiviitamaki merged 1 commit intodevelopfrom
task/TP-1989_group-admin-notifications

Conversation

@jussiviitamaki
Copy link
Contributor

@jussiviitamaki jussiviitamaki commented Dec 23, 2025

Actions necessary for applying the changes: (for example composer install; drush updb; drush cim)

lando drush cr

Testing instructions:

  • lando test /public/modules/custom/service_manual_workflow/tests/src/Kernel/ServiceStateChangedNotificationSubscriberTest.php
  • Confirm tests pass
  • As service provider updater create or edit draft service to service group
  • Move created service to ready to publish
  • Confirm service ready to publish mail was sent to all parent groups admins

Review checklist:

  • The code conforms to Drupal coding standards
  • I have reviewed the code for security and quality issues
  • I have tested the code with the proper user roles
  • I have moved the ticket to the approval lane, added testing instructions and assigned it to the product owner.

@jussiviitamaki jussiviitamaki merged commit 24f1eb8 into develop Jan 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants