Added a new option in workflow notificationSettings to ensure that notifications are also sent to inactive users.

[rahul-pim-dev]

Added a new option in workflow notificationSettings to ensure that notifications are also sent to inactive users.

see : #16867

[github-actions]

Review Checklist

• Target branch (11.2 for bug fixes, others 11.x)
• Tests (if it's testable code, there should be a test for it - get help)
• Docs (every functionality needs to be documented, see here)
• Migration incl. install.sql (e.g. if the database schema changes, ...)
• Upgrade notes (deprecations, important information, migration hints, ...)
• Label
• Milestone

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarCloud

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[markus-moser]

@rahul-pim-dev Thanks a lot, could you please sign the CLA?

I would call the current behaviour more like a bug. The workflow engine should not send notifications to inactive users. I would say let's not make this configurable. I would call this more like a "security leak". When someone deactivates a user this user should not get any notifications anymore (and therefore receive potentially internal information). If someone wants to send notifications to users without real access to the Pimcore backend it would be possible to add a user without rights but activate it.