Closed amorask-bitwarden closed 3 days ago
Attention: Patch coverage is 0%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 29.31%. Comparing base (
679c25b
) to head (97f3ae7
).
Files | Patch % | Lines |
---|---|---|
...roviders/clients/create-client-dialog.component.ts | 0.00% | 4 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One โ Scan Summary & Details โ ce68d091-729a-471e-b410-a1b1a673761a
Severity | Issue | Source File / Package | Checkmarx Insight |
---|---|---|---|
![]() |
Angular_Improper_Type_Pipe_Usage | /bitwarden_license/bit-web/src/app/admin-console/providers/providers-layout.component.html: 49 | Attack Vector |
๐๏ธ Tracking
https://bitwarden.atlassian.net/browse/AC-2803
๐ Objective
Updates the Provider Portal to dynamically display or hide the UI elements related to a provider's subscription discount based on whether one exists or not. Previously, some of this was hard-coded as we always expected the guaranteed 35% discount.
๐ธ Screenshots
https://github.com/bitwarden/clients/assets/144709477/55895b34-480d-4000-8326-0fa5e84d44e1
โฐ Reminders before review
๐ฆฎ Reviewer guidelines
:+1:
) or similar for great changes:memo:
) or โน๏ธ (:information_source:
) for notes or general info:question:
) for questions:thinking:
) or ๐ญ (:thought_balloon:
) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion:art:
) for suggestions / improvements:x:
) or โ ๏ธ (:warning:
) for more significant problems or concerns needing attention:seedling:
) or โป๏ธ (:recycle:
) for future improvements or indications of technical debt:pick:
) for minor or nitpick changes