bitwarden / ios

Bitwarden mobile app for iOS.
https://bitwarden.com
GNU General Public License v3.0
140 stars 12 forks source link

[PM-9842] Show expired link screen #862

Closed andrebispo5 closed 3 weeks ago

andrebispo5 commented 3 weeks ago

๐ŸŽŸ๏ธ Tracking

https://bitwarden.atlassian.net/browse/PM-9842

๐Ÿ“” Objective

Show expired link screen when the token validation fails on CompleteRegistration appear.

๐Ÿ“ธ Screenshots

https://github.com/user-attachments/assets/5cc3773a-3ac0-46c0-9d3a-ba2750162567

โฐ Reminders before review

๐Ÿฆฎ Reviewer guidelines

fedemkr commented 3 weeks ago

@andrebispo5 Is it ok that the user can't press the "X" button? Or I saw this wrongly?

andrebispo5 commented 3 weeks ago

@andrebispo5 Is it ok that the user can't press the "X" button? Or I saw this wrongly?

We can press, I was just hovering the X. It goes to landing.

github-actions[bot] commented 3 weeks ago

Logo Checkmarx One โ€“ Scan Summary & Details โ€“ 043a36a4-ba86-474a-bb60-3d1fc3c4b79f

No New Or Fixed Issues Found

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.63%. Comparing base (c11be7e) to head (04df2b3). Report is 2 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #862 +/- ## ======================================= Coverage 88.63% 88.63% ======================================= Files 615 615 Lines 30592 30605 +13 ======================================= + Hits 27115 27128 +13 Misses 3477 3477 ```

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