Skip to content

bugfix/FOUR-28729: [48874] Ellucian - Email Notifications to "Assigned User" Are Suppressed When the Same User Initiates the Request or Is Assigned to Consecutive Tasks#8848

Open
taniachoquecondori wants to merge 2 commits into
developfrom
bugfix/FOUR-28729
Open

bugfix/FOUR-28729: [48874] Ellucian - Email Notifications to "Assigned User" Are Suppressed When the Same User Initiates the Request or Is Assigned to Consecutive Tasks#8848
taniachoquecondori wants to merge 2 commits into
developfrom
bugfix/FOUR-28729

Conversation

@taniachoquecondori
Copy link
Copy Markdown
Contributor

@taniachoquecondori taniachoquecondori commented Jun 2, 2026 •
edited
Loading

Issue & Reproduction Steps

Emails to Assigned User are skipped when the assignee is the same user on consecutive tasks or the request starter. Repro: 2 tasks → both User A → notification on task 2 does not send.

Solution

  • Core: notifications_send_to_same_user config (env NOTIFICATIONS_SEND_TO_SAME_USER).
  • Connector-send-email: only skip same-user when config is false.

How to Test

  • NOTIFICATIONS_SEND_TO_SAME_USER=true → email sends on each assigned task.
  • false → old skip behavior.

Related Tickets & Packages

ci:deploy
ci:connector-send-email:bugfix/FOUR-28729

@Kookster310
Copy link
Copy Markdown
Contributor

Kookster310 commented Jun 2, 2026

QA server K8S was successfully deployed https://ci-23e6edd00c.engk8s.processmaker.net

pmPaulis
pmPaulis reviewed Jun 2, 2026
View reviewed changes
Comment thread config/app.php Outdated
@pmPaulis
Copy link
Copy Markdown
Contributor

pmPaulis commented Jun 2, 2026 

Notifications (Tests\Unit\Notifications)
 ✘ Assigned user notification respects same user config
   │
   │ ArgumentCountError: Too few arguments to function Tests\Unit\NotificationsTest::testAssignedUserNotificationRespectsSameUserConfig(), 0 passed in /opt/processmaker/vendor/phpunit/phpunit/src/Framework/TestCase.php on line 1332 and exactly 4 expected
   │
   │ /opt/packages/connector-send-email/tests/unit/NotificationsTest.php:28
   │

@pmPaulis pmPaulis requested a review from JimmyQuiroga007 June 2, 2026 14:40
JimmyQuiroga007
JimmyQuiroga007 approved these changes Jun 2, 2026
View reviewed changes
Copy link
Copy Markdown

@JimmyQuiroga007 JimmyQuiroga007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks fine

@processmaker-sonarqube
Copy link
Copy Markdown

processmaker-sonarqube Bot commented Jun 2, 2026

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@Kookster310
Copy link
Copy Markdown
Contributor

Kookster310 commented Jun 2, 2026

QA server K8S was successfully deployed https://ci-23e6edd00c.engk8s.processmaker.net

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pmPaulis pmPaulis pmPaulis left review comments

@JimmyQuiroga007 JimmyQuiroga007 JimmyQuiroga007 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@taniachoquecondori @Kookster310 @pmPaulis @JimmyQuiroga007