Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CustomerNotificationsDisabled option #380

Closed
wants to merge 4 commits into from

Conversation

pboguslawski
Copy link
Contributor

Proposed change

This mod introduces new SysConfig parameter
CustomerNotificationsDisabled that allowes one to disable sending
notifications to customers with Ticket Notifications (extra
protection against customer spamming).

Type of change

  • '1 - 🚀 feature' - New feature (which adds functionality to an existing integration)

Additional information

Author-Change-Id: IB#1050079

Checklist

  • The code change is tested and works locally.(❗)
  • There is no commented out code in this PR.(❕)
  • You improved or added new unit tests.(❕)
  • Local ZnunyCodePolicy passed.(❕)
  • Local UnitTests / Selenium passed.(❕)
  • GitHub workflow CI (UnitTests / Selenium) passed.(❗)

This mod introduces new SysConfig parameter
CustomerNotificationsDisabled that allowes one to disable sending
notifications to customers with Ticket Notifications (extra
protection against customer spamming).

Author-Change-Id: IB#1050079
Fixes: d5e071442e230ea096a26ef310ff848a54c9121b
Author-Change-Id: IB#1050079
Author-Change-Id: IB#1050079
Author-Change-Id: IB#1050079
@dennykorsukewitz dennykorsukewitz added the 1 - 🚀 feature New feature or request label Jun 9, 2023
@rkaldung
Copy link
Member

@pboguslawski We do not accept this change. The same behavior can be achieved by not selecting the customer user as a recipient.

@rkaldung rkaldung closed this Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 - 🚀 feature New feature or request
Development

Successfully merging this pull request may close these issues.

3 participants