Skip to content
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

Update autofill dialog copy, changing "logins" to "passwords" #3867

Conversation

CDRussell
Copy link
Member

@CDRussell CDRussell commented Nov 15, 2023

Task/Issue URL: https://app.asana.com/0/0/1205900373687630/f

Description

Updates copy, preferring to use terminology "password" instead of "login" in the autofill dialogs.

Includes translations.

Steps to test this PR

  • QA optional

@CDRussell CDRussell force-pushed the feature/craig/update_copy_for_autofill_dialogs branch from 4e0ad24 to 6f4923e Compare November 15, 2023 10:44
@CDRussell CDRussell marked this pull request as ready for review November 15, 2023 10:46
@joshliebe joshliebe self-assigned this Nov 15, 2023
Copy link
Contributor

@joshliebe joshliebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@CDRussell CDRussell merged commit ab8831e into feature/craig/autofill_enabled_by_default Nov 15, 2023
@CDRussell CDRussell deleted the feature/craig/update_copy_for_autofill_dialogs branch November 15, 2023 11:08
CDRussell added a commit that referenced this pull request Nov 15, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

Task/Issue URL: https://app.asana.com/0/0/1205900373687630/f 

### Description
Updates copy, preferring to use terminology "password" instead of
"login" in the autofill dialogs.

Includes translations.

### Steps to test this PR
- QA optional

Co-authored-by: root <root@10.100.11.5>
CDRussell added a commit that referenced this pull request Nov 15, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

Task/Issue URL: https://app.asana.com/0/0/1205900373687630/f

### Description
Updates copy, preferring to use terminology "password" instead of
"login" in the autofill dialogs.

Includes translations.

### Steps to test this PR
- QA optional

Co-authored-by: root <root@10.100.11.5>
CDRussell added a commit that referenced this pull request Nov 15, 2023
<!--
Note: This checklist is a reminder of our shared engineering
expectations.
The items in Bold are required
If your PR involves UI changes:
1. Upload screenshots or screencasts that illustrate the changes before
/ after
2. Add them under the UI changes section (feel free to add more columns
if needed)
If your PR does not involve UI changes, you can remove the **UI
changes** section

At a minimum, make sure your changes are tested in API 23 and one of the
more recent API levels available.
-->

Task/Issue URL: https://app.asana.com/0/0/1205900373687630/f

### Description
Updates copy, preferring to use terminology "password" instead of
"login" in the autofill dialogs.

Includes translations.

### Steps to test this PR
- QA optional

Co-authored-by: root <root@10.100.11.5>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants