Skip to content

FOUR-32617: variable to disable the sendEmail by task - #8982

Open
pmPaulis wants to merge 2 commits into
developfrom
FOUR-32617
Open

pmPaulis wants to merge 2 commits into
developfrom
FOUR-32617

Conversation

@pmPaulis

@pmPaulis pmPaulis commented Aug 10, 2026 •

Copy link
Copy Markdown
Contributor

Create a variable to turn off notifications by defaultthe steps above

Related Tickets & Packages

Code Review Checklist

  • I have pulled this code locally and tested it on my instance, along with any associated packages.
  • This code adheres to ProcessMaker Coding Guidelines.
  • This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
  • This solution fixes the bug reported in the original ticket.
  • This solution does not alter the expected output of a component in a way that would break existing Processes.
  • This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
  • This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
  • This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
  • This ticket conforms to the PRD associated with this part of ProcessMaker.

ci:deploy
ci:connector-send-email:FOUR-32617

@processmaker-sonarqube

Copy link
Copy Markdown

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

@pmPaulis pmPaulis self-assigned this Aug 10, 2026
@pmPaulis
pmPaulis requested a review from caleeli August 12, 2026 21:37
@vladyrichter

Copy link
Copy Markdown

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

@cursor

cursor Bot commented Sep 29, 2026 •

Copy link
Copy Markdown

PR Summary

Low Risk
Config-only change with default true; risk is low until other code reads the flag and changes notification defaults.

Overview
Adds a new EMAIL_TASKS_DEFAULT_ENABLED env-backed setting (config('app.email_tasks_default_enabled')) so deployments can control whether task email notifications are on by default, without changing current behavior when unset (defaults to true).

This supports FOUR-32617 (turning off task sendEmail notifications by default via environment configuration). No application code in this diff reads the new key yet—expect the send-email connector or follow-up changes to honor it.

Reviewed by Cursor Bugbot for commit 90ada66. Bugbot is set up for automated code reviews on this repo. Configure here.

@decisions-sonarqube

Copy link
Copy Markdown

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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants