Bug 1799622 - A password deleted in the password manager, is still visible in the list afterwards

gitstart commented 1 year ago

The PR fixes saved Logins not updating immediately a login is deleted

Fix description

Navigate back to the saved login screen, rather than just popping back stack

Issue video

fnx-58-issue.webm

Demo video after fix

fnx-58-fix.webm

Pull Request checklist

[ ] Tests: This PR includes thorough tests or an explanation of why it does not
[x] Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not
[ ] Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features. In addition, it includes a screenshot of a successful accessibility scan to ensure no new defects are added to the product.

QA

[x] QA Needed
To download an APK when reviewing a PR (after all CI tasks finished running):

Click on Checks at the top of the PR page.
Click on the firefoxci-taskcluster group on the left to expand all tasks.
Click on the build-debug task.
Click on View task in Taskcluster in the new DETAILS section.
The APK links should be on the right side of the screen, named for each CPU architecture.

GitHub Automation

Used by GitHub Actions. Fixes https://bugzilla.mozilla.org/show_bug.cgi?id=1799622 and https://bugzilla.mozilla.org/show_bug.cgi?id=1812431 (duplicate)

firefoxci-taskcluster[bot] commented 1 year ago

No Taskcluster jobs started for this pull request

```js The `allowPullRequests` configuration for this repository (in `.taskcluster.yml` on the default branch) does not allow starting tasks for this pull request. ```

csadilek commented 1 year ago

Hey @gitstart, Thank you so much for your contribution. It’s much appreciated. Unfortunately, we are in the process of moving this repository to firefox-android early next week, and won’t be able to review or land any PRs until after the migration. Please feel free to reopen a new PR against the new repo once migration is complete. You will have to clone the new repository and reapply your changes. Thank you again, and sorry for the inconvenience.

For more information, please see https://github.com/mozilla-mobile/fenix/issues/26855.

gitstart commented 1 year ago

Hey @gitstart, Thank you so much for your contribution. It’s much appreciated. Unfortunately, we are in the process of moving this repository to firefox-android early next week, and won’t be able to review or land any PRs until after the migration. Please feel free to reopen a new PR against the new repo once migration is complete. You will have to clone the new repository and reapply your changes. Thank you again, and sorry for the inconvenience.

For more information, please see #26855.

Okay @csadilek, totally understandable.. will definitely repost the PRs when the MonoRepo is ready. Thank you.

mozilla-mobile / fenix