Closed ike-kottlowski closed 3 months ago
Attention: Patch coverage is 14.28571%
with 6 lines
in your changes missing coverage. Please review.
Project coverage is 29.46%. Comparing base (
c01f6be
) to head (2805e9f
).
Files | Patch % | Lines |
---|---|---|
...uth/settings/two-factor-authenticator.component.ts | 16.66% | 4 Missing and 1 partial :warning: |
...src/app/auth/settings/two-factor-base.component.ts | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One â Scan Summary & Details â 0cd88ac1-021d-4dca-94fe-f6a4b8380eee
đī¸ Tracking
PM-8938
đ Objective
Validation on token field and error toasts were failing silently when users try to add improper token values. This communicates user friendly validation information to the end user.
đ¸ Screenshots
https://github.com/bitwarden/clients/assets/137194738/a0902cdf-22b0-4fde-b3b4-c6cf34ca5ca4
https://github.com/bitwarden/clients/assets/137194738/18764d31-4603-45c5-9942-55573ef8236a
â° 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