-
Notifications
You must be signed in to change notification settings - Fork 160
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Change wording to "Verify identity" #3751
Conversation
📱 Scan the QR code below to install the build (arm64 only) for this PR. |
Quality Gate passedIssues Measures |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #3751 +/- ##
========================================
Coverage 83.09% 83.10%
========================================
Files 1770 1771 +1
Lines 44378 44415 +37
Branches 5202 5207 +5
========================================
+ Hits 36878 36909 +31
- Misses 5669 5674 +5
- Partials 1831 1832 +1 ☔ View full report in Codecov by Sentry. |
@@ -106,7 +106,7 @@ fun UserProfileView( | |||
private fun VerifyUserSection(state: UserProfileState) { | |||
if (state.isVerified.dataOrNull() == false) { | |||
ListItem( | |||
headlineContent = { Text(stringResource(R.string.screen_room_member_details_verify_button_title, state.userName ?: state.userId)) }, | |||
headlineContent = { Text(stringResource(CommonStrings.common_verify_identity)) }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be action.verify_identity
as verify is a verb here?
See the discussion here: https://www.figma.com/design/0oUTCbYhROEDprSp7bdWhx?node-id=896-11141#1004391159