Skip to content

Commit 3c6a388

Browse files
committed
profile: Display user email
Fixes: zulip#291
1 parent 6324bf3 commit 3c6a388

File tree

2 files changed

+7
-2
lines changed

2 files changed

+7
-2
lines changed

lib/widgets/profile.dart

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,10 @@ class ProfilePage extends StatelessWidget {
5050
textAlign: TextAlign.center,
5151
style: _TextStyles.primaryFieldText
5252
.merge(weightVariableTextStyle(context, wght: 700))),
53-
// TODO(#291) render email field
53+
if (user.deliveryEmail != null)
54+
Text(user.deliveryEmail!,
55+
textAlign: TextAlign.center,
56+
style: _TextStyles.primaryFieldText),
5457
Text(roleToLabel(user.role, zulipLocalizations),
5558
textAlign: TextAlign.center,
5659
style: _TextStyles.primaryFieldText),

test/widgets/profile_test.dart

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,12 +71,14 @@ void main() {
7171

7272
group('ProfilePage', () {
7373
testWidgets('page builds; profile page renders', (WidgetTester tester) async {
74-
final user = eg.user(userId: 1, fullName: 'test user');
74+
final user = eg.user(userId: 1, fullName: 'test user',
75+
deliveryEmail: '[email protected]');
7576

7677
await setupPage(tester, users: [user], pageUserId: user.userId);
7778

7879
check(because: 'find user avatar', find.byType(Avatar).evaluate()).length.equals(1);
7980
check(because: 'find user name', find.text('test user').evaluate()).isNotEmpty();
81+
check(because: 'find user delivery email', find.text('[email protected]').evaluate()).isNotEmpty();
8082
});
8183

8284
testWidgets('page builds; profile page renders with profileData', (WidgetTester tester) async {

0 commit comments

Comments
 (0)