Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[Bug] Fennec migration: "Looking for your logins?" message card is displayed after entering primary password and returning to home screen #14676

Closed
ebalazs-sv opened this issue Sep 3, 2020 · 1 comment
Assignees
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:Migration needs:triage Issue needs triage S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist

Comments

@ebalazs-sv
Copy link

ebalazs-sv commented Sep 3, 2020

Preconditions

On Fennec create a Primary(Master) password.
Save some logins on Fennec with the primary password enabled.

Steps to reproduce

  1. Migrate Fennec to Fenix;
  2. Go to home screen and tap the "Enter master password" button from "Looking for your logins?" messaging card;
  3. Type your primary password and tap "Continue and transfer";
  4. Tap "View saved longins" from the Transfer successful pop up;
  5. Go back to home screen;

Expected behavior

"Looking for your logins?" message card is dismissed.

Actual behavior

"Looking for your logins?" message card is still displayed and the primary password can be added multiple times.

20200903_134934

Device information

  • Android device: Samsung Galaxy Tab S3 (Android 8), Nexus 5 (Android 5);
  • Fenix version: Beta 81.1.0-beta.2 (Build #2015761657), GV 81.0, from 9/2.

┆Issue is synchronized with this Jira Task

@ebalazs-sv ebalazs-sv added 🐞 bug Crashes, Something isn't working, .. Feature:Migration S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist labels Sep 3, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Sep 3, 2020
@ekager ekager self-assigned this Sep 3, 2020
ekager added a commit to ekager/fenix that referenced this issue Sep 3, 2020
@ekager ekager added the eng:qa:needed QA Needed label Sep 9, 2020
@ebalazs-sv
Copy link
Author

Verified as fixed on Beta 81.1.1-beta.3 (Build #2015763233) GV 81.0 from 9/11 with Samsung Galaxy Tab S3 (Android 8).
I will close this issue and remove the qa:needed label.

@ebalazs-sv ebalazs-sv added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Sep 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:Migration needs:triage Issue needs triage S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist
Projects
None yet
Development

No branches or pull requests

2 participants