notifications: Shorten string for redacted content.

Replace the long string for organisations that have notification
body/content disabled (settings.PUSH_NOTIFICATION_REDACT_CONTENT
set to true) with "New message".

This allows more of the limited space on the mobile device screen to
be used for additional messages rather than this verbose content.

Fixes #29152
This commit is contained in:
Aditya Bajaj 2024-03-23 00:37:51 +08:00 committed by GitHub
parent 92fa9ee78c
commit a20b454ebb
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 3 additions and 9 deletions

View File

@ -922,13 +922,7 @@ def get_mobile_push_content(rendered_content: str) -> str:
return plain_text
if settings.PUSH_NOTIFICATION_REDACT_CONTENT:
return (
"*"
+ _(
"This organization has disabled including message content in mobile push notifications"
)
+ "*"
)
return _("New message")
elem = lxml.html.fragment_fromstring(rendered_content, create_parent=True)
change_katex_to_raw_latex(elem)

View File

@ -4109,7 +4109,7 @@ class TestGetAPNsPayload(PushNotificationTest):
"alert": {
"title": "Cordelia, Lear's daughter, King Hamlet, Othello, the Moor of Venice",
"subtitle": "King Hamlet:",
"body": "*This organization has disabled including message content in mobile push notifications*",
"body": "New message",
},
"sound": "default",
"badge": 0,
@ -4305,7 +4305,7 @@ class TestGetGCMPayload(PushNotificationTest):
"event": "message",
"zulip_message_id": message.id,
"time": datetime_to_timestamp(message.date_sent),
"content": "*This organization has disabled including message content in mobile push notifications*",
"content": "New message",
"content_truncated": False,
"server": settings.EXTERNAL_HOST,
"realm_id": hamlet.realm.id,