Skip to content

[PM-3803] Adding AutomationIDs for Set/Update password pages#2748

Merged
ifernandezdiaz merged 2 commits intomasterfrom
PM-3803/adding-automationids-for-set-password
Sep 7, 2023
Merged

[PM-3803] Adding AutomationIDs for Set/Update password pages#2748
ifernandezdiaz merged 2 commits intomasterfrom
PM-3803/adding-automationids-for-set-password

Conversation

@ifernandezdiaz
Copy link
Contributor

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc.)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Adding more AutomationIDs to continue improving our mobile automation test suite

Code changes

Some missing AutomationIDs were added to Set/UpdatePassword XAML files

  • SetPasswordPage.xaml: Adding missing IDs for entry fields
  • UpdateTempPasswordPage.xaml: Adding missing IDs for entry fields

Before you submit

  • Please check for formatting errors (dotnet format --verify-no-changes) (required)
  • Please add unit tests where it makes sense to do so (encouraged but not required)
  • If this change requires a documentation update - notify the documentation team
  • If this change has particular deployment requirements - notify the DevOps team

@ifernandezdiaz ifernandezdiaz merged commit 3552616 into master Sep 7, 2023
@ifernandezdiaz ifernandezdiaz deleted the PM-3803/adding-automationids-for-set-password branch September 7, 2023 14:24
@bitwarden-bot
Copy link

Logo
Checkmarx One – Scan Summary & Detailsbe577f84-babb-4f1d-8c4a-8c1297e68773

No New Or Fixed Issues Found

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants