PLT-7130 adding more generic push notificaiton message. (mattermost#7044

) * PLT-7130 adding more generic push notificaiton message. * Fixing strings
stevepartridge · Aug 3, 2017 · 753d1e5 · 753d1e5
1 parent 59b3f98
commit 753d1e5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/components/admin_console/push_settings.jsx b/components/admin_console/push_settings.jsx
@@ -207,7 +207,8 @@ export default class PushSettings extends AdminSettings {
  <DropdownSetting
  id='pushNotificationContents'
  values={[
- {value: 'generic', text: Utils.localizeMessage('admin.email.genericPushNotification', 'Send generic description with user and channel names')},
+ {value: 'generic_no_channel', text: Utils.localizeMessage('admin.email.genericNoChannelPushNotification', '"Send generic description with only sender name')},
+ {value: 'generic', text: Utils.localizeMessage('admin.email.genericPushNotification', 'Send generic description with sender and channel names')},
  {value: 'full', text: Utils.localizeMessage('admin.email.fullPushNotification', 'Send full message snippet')}
  ]}
  label={
@@ -222,8 +223,7 @@ export default class PushSettings extends AdminSettings {
  helpText={
  <FormattedHTMLMessage
  id='admin.email.pushContentDesc'
- defaultMessage='Selecting "Send generic description with user and channel names" provides push notifications with generic messages, including names of users and channels but no specific details from the message text.<br /><br />
- Selecting "Send full message snippet" sends excerpts from messages triggering notifications with specifics and may include confidential information sent in messages. If your Push Notification Service is outside your firewall, it is HIGHLY RECOMMENDED this option only be used with an "https" protocol to encrypt the connection.'
+ defaultMessage='"Send generic description with only sender name" includes only the name of the person who sent the message in push notifications, with no information about channel name or message contents.<br /><br />"Send generic description with sender and channel names" includes the name of the person who sent the message and the channel it was sent in, but not the message text.<br /><br />"Send full message snippet" includes a message excerpt in push notifications, which may contain confidential information sent in messages. If your Push Notification Service is outside your firewall, it is *highly recommended* this option only be used with an "https" protocol to encrypt the connection.'
  />
  }
  />

diff --git a/i18n/en.json b/i18n/en.json
@@ -297,7 +297,8 @@
  "admin.email.enableEmailBatchingDesc": "When true, users will have email notifications for multiple direct messages and mentions combined into a single email. Batching will occur at a default interval of 15 minutes, configurable in Account Settings > Notifications.",
  "admin.email.enableEmailBatchingTitle": "Enable Email Batching:",
  "admin.email.fullPushNotification": "Send full message snippet",
- "admin.email.genericPushNotification": "Send generic description with user and channel names",
+ "admin.email.genericNoChannelPushNotification": "Send generic description with only sender name",
+ "admin.email.genericPushNotification": "Send generic description with sender and channel names",
  "admin.email.inviteSaltDescription": "32-character salt added to signing of email invites. Randomly generated on install. Click \"Regenerate\" to create new salt.",
  "admin.email.inviteSaltExample": "E.g.: \"bjlSR4QqkXFBr7TP4oDzlfZmcNuH9Yo\"",
  "admin.email.inviteSaltTitle": "Email Invite Salt:",
@@ -320,7 +321,7 @@
  "admin.email.passwordSaltDescription": "32-character salt added to signing of password reset emails. Randomly generated on install. Click \"Regenerate\" to create new salt.",
  "admin.email.passwordSaltExample": "E.g.: \"bjlSR4QqkXFBr7TP4oDzlfZmcNuH9Yo\"",
  "admin.email.passwordSaltTitle": "Password Reset Salt:",
- "admin.email.pushContentDesc": "Selecting \"Send generic description with user and channel names\" provides push notifications with generic messages, including names of users and channels but no specific details from the message text.<br /><br />Selecting \"Send full message snippet\" sends excerpts from messages triggering notifications with specifics and may include confidential information sent in messages. If your Push Notification Service is outside your firewall, it is HIGHLY RECOMMENDED this option only be used with an \"https\" protocol to encrypt the connection.",
+ "admin.email.pushContentDesc": "\"Send generic description with only sender name\" includes only the name of the person who sent the message in push notifications, with no information about channel name or message contents.<br /><br />\"Send generic description with sender and channel names\" includes the name of the person who sent the message and the channel it was sent in, but not the message text.<br /><br />\"Send full message snippet\" includes a message excerpt in push notifications, which may contain confidential information sent in messages. If your Push Notification Service is outside your firewall, it is *highly recommended* this option only be used with an \"https\" protocol to encrypt the connection.",
  "admin.email.pushContentTitle": "Push Notification Contents:",
  "admin.email.pushDesc": "Typically set to true in production. When true, Mattermost attempts to send iOS and Android push notifications through the push notification server.",
  "admin.email.pushOff": "Do not send push notifications",