Closed andrebispo5 closed 3 weeks ago
Attention: Patch coverage is 95.77465%
with 3 lines
in your changes missing coverage. Please review.
Project coverage is 88.63%. Comparing base (
d0dc545
) to head (1470573
). Report is 3 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
...leteRegistration/ExpiredLink/ExpiredLinkView.swift | 93.75% | 2 Missing :warning: |
...egistration/ExpiredLink/ExpiredLinkProcessor.swift | 90.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 โ a6ed6ae1-90ac-4b59-97b0-338ca0e68852
โ๏ธ Can the name of the PR (/message on the eventual merge commit) be updated with [PM-9842]
?
โ๏ธ Can the name of the PR (/message on the eventual merge commit) be updated with
[PM-9842]
?
For sure, totally miss that! Thank you :)
๐๏ธ Tracking
https://bitwarden.atlassian.net/browse/PM-9842
๐ Objective
Add necessary classes for the expired link UI.
๐ธ Screenshots
โฐ 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