search

bitwarden / passwordless-server

Bitwarden Passwordless.dev infrastructure/backend (API, database, Docker, etc).
https://bitwarden.com/
Other
86 stars 28 forks source link

PAS-567 | Cannot delete credentials when admin has multiple credentials #708

Closed jonashendrickx closed 2 months ago

jonashendrickx commented 2 months ago

Ticket

Description

Shape

Screenshots

Checklist

I did the following to ensure that my changes were tested thoroughly:

I did the following to ensure that my changes do not introduce security vulnerabilities:

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 0% with 9 lines in your changes missing coverage. Please review.

Project coverage is 35.15%. Comparing base (063515c) to head (c1fb560). Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...c/AdminConsole/Components/Shared/Credentials.razor 0.00% 5 Missing :warning:
...dminConsole/Components/Shared/Credentials.razor.cs 0.00% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #708 +/- ## ======================================= Coverage 35.15% 35.15% ======================================= Files 574 574 Lines 31156 31156 Branches 930 930 ======================================= Hits 10953 10953 Misses 20057 20057 Partials 146 146 ```

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