Closed djsmith85 closed 3 months ago
Attention: Patch coverage is 0%
with 4 lines
in your changes missing coverage. Please review.
Project coverage is 29.33%. Comparing base (
8df054c
) to head (028c37e
). Report is 2 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
apps/web/src/app/tools/send/access.component.ts | 0.00% | 2 Missing :warning: |
apps/web/src/app/tools/send/send.component.ts | 0.00% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One â Scan Summary & Details â 59a2c2f7-3e5b-4e47-8f8f-57b1429cd39f
đī¸ Tracking
https://bitwarden.atlassian.net/browse/PM-9250
đ Objective
Preparing to re-use the existing icons on other clients (namely browser for the extension UI refresh)
â° 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