Skip to content

Commit 2393a19

Browse files
rajveermalviyagnprice
authored andcommitted
notif [nfc]: Use localization for self user display name
1 parent 1596a1a commit 2393a19

File tree

3 files changed

+8
-2
lines changed

3 files changed

+8
-2
lines changed

assets/l10n/app_en.arb

+4
Original file line numberDiff line numberDiff line change
@@ -479,5 +479,9 @@
479479
"senderFullName": {"type": "String", "example": "Alice"},
480480
"numOthers": {"type": "int", "example": "4"}
481481
}
482+
},
483+
"notifSelfUser": "You",
484+
"@notifSelfUser": {
485+
"description": "Display name for the user themself, to show after replying in an Android notification"
482486
}
483487
}

lib/notifications/display.dart

+1-1
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ class NotificationDisplayManager {
108108
messagingStyle = MessagingStyle(
109109
user: Person(
110110
key: _personKey(data.realmUri, data.userId),
111-
name: 'You'), // TODO(i18n)
111+
name: zulipLocalizations.notifSelfUser),
112112
messages: [],
113113
isGroupConversation: switch (data.recipient) {
114114
FcmMessageStreamRecipient() => true,

test/notifications/display_test.dart

+3-1
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ import 'package:flutter_test/flutter_test.dart';
1111
import 'package:zulip/api/model/model.dart';
1212
import 'package:zulip/api/notifications.dart';
1313
import 'package:zulip/host/android_notifications.dart';
14+
import 'package:zulip/model/localizations.dart';
1415
import 'package:zulip/model/narrow.dart';
1516
import 'package:zulip/model/store.dart';
1617
import 'package:zulip/notifications/display.dart';
@@ -76,6 +77,7 @@ MessageFcmMessage messageFcmMessage(
7677

7778
void main() {
7879
TestZulipBinding.ensureInitialized();
80+
final zulipLocalizations = GlobalLocalizations.zulipLocalizations;
7981

8082
Future<void> init() async {
8183
addTearDown(testBinding.reset);
@@ -150,7 +152,7 @@ void main() {
150152
..user.which((it) => it
151153
..iconBitmap.isNull()
152154
..key.equals(expectedSelfUserKey)
153-
..name.equals('You')) // TODO(i18n)
155+
..name.equals(zulipLocalizations.notifSelfUser))
154156
..isGroupConversation.equals(expectedIsGroupConversation)
155157
..conversationTitle.equals(expectedTitle)
156158
..messages.deepEquals(messageStyleMessagesChecks))

0 commit comments

Comments
 (0)