From e1826b661bad2c52a6fad1e2a7b2873968b8c888 Mon Sep 17 00:00:00 2001
From: Michael <Chefkeks@users.noreply.github.com>
Date: Mon, 5 Oct 2015 12:21:16 +0200
Subject: [PATCH] UI: Assignment alerts do NOT include teams
Hi guys,
had this issue in my mind from before my vacation. Someone over in the forum had the issue that even though he checked "Assigned Agent / Team" the team never got notified. After some research and a closer look I guess Neil found out that he needed to check "Team Lead" and/or "Team members" to enable alerts for the team as well.
So in this case the wording "Assigned Agent / Team" is a bit misleading and I suggest to change it to just "Assigned Agent" or whatever you prefer ;)
Just made this pull request, so that you don't need to search long and it's (hopefully) easier to understand which "Assigned Agent / Team" is meant since it's there several times at the alert settings page.
Cheers,
Michael
PS: Also I don't know why the first line appears as "removed" and "added" ... didn't change anything here, but anyway, line 126 is the important one ;)
---
include/staff/settings-alerts.inc.php | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/staff/settings-alerts.inc.php b/include/staff/settings-alerts.inc.php
index b08cdbe4d..4b69e1803 100644
--- a/include/staff/settings-alerts.inc.php
+++ b/include/staff/settings-alerts.inc.php
@@ -1,4 +1,4 @@
-<table class="form_table settings_table" width="940" border="0" cellspacing="0" cellpadding="2">
+<table class="form_table settings_table" width="940" border="0" cellspacing="0" cellpadding="2">
<tbody>
<tr><th><em><b><?php echo __('New Ticket Alert'); ?></b>:
<i class="help-tip icon-question-sign" href="#ticket_alert"></i>
@@ -123,7 +123,7 @@
<tr>
<td>
<input type="checkbox" name="assigned_alert_staff" <?php echo
- $config['assigned_alert_staff']?'checked':''; ?>> <?php echo __('Assigned Agent / Team'); ?>
+ $config['assigned_alert_staff']?'checked':''; ?>> <?php echo __('Assigned Agent'); ?>
</td>
</tr>
<tr>
--
GitLab