bitwarden / android

Bitwarden mobile app for Android.
https://bitwarden.com
GNU General Public License v3.0
6.55k stars 819 forks source link

PM-14963: Add toast when login via device succeeds #4351

Closed david-livefront closed 1 day ago

david-livefront commented 1 day ago

๐ŸŽŸ๏ธ Tracking

PM-14963

๐Ÿ“” Objective

This PR adds a toast when you successfully login with device.

โฐ Reminders before review

๐Ÿฆฎ Reviewer guidelines

github-actions[bot] commented 1 day ago

Logo Checkmarx One โ€“ Scan Summary & Details โ€“ cc5e3ae0-e8f3-4687-9968-a6ea0b81e5a2

New Issues

Severity Issue Source File / Package Checkmarx Insight
MEDIUM Privacy_Violation /app/src/main/java/com/x8bit/bitwarden/ui/vault/feature/item/VaultItemViewModel.kt: 875 Attack Vector

Fixed Issues

Severity Issue Source File / Package
MEDIUM Unpinned Actions Full Length Commit SHA /crowdin-pull.yml: 26
david-livefront commented 1 day ago

Thanks @dseverns-livefront

codecov[bot] commented 1 day ago

Codecov Report

Attention: Patch coverage is 80.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 89.00%. Comparing base (d4033a7) to head (998fee7). Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
...h/feature/loginwithdevice/LoginWithDeviceScreen.kt 50.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #4351 +/- ## ======================================= Coverage 89.00% 89.00% ======================================= Files 452 452 Lines 39155 39157 +2 Branches 5524 5524 ======================================= + Hits 34848 34852 +4 + Misses 2379 2377 -2 Partials 1928 1928 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


๐Ÿšจ Try these New Features: