Closed KatherineInCode closed 5 months ago
Checkmarx One â Scan Summary & Details â 62a42459-1b9f-4bf5-aa77-7b96ca56a111
Attention: Patch coverage is 57.14286%
with 3 lines
in your changes missing coverage. Please review.
Project coverage is 61.78%. Comparing base (
1b5a279
) to head (f273fb6
).
Files | Patch % | Lines |
---|---|---|
...gs/Settings/ImportItems/ImportItemsProcessor.swift | 0.00% | 2 Missing :warning: |
...ared/Core/Vault/Models/Enum/ImportFormatType.swift | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
đ Objective
This updates the 2FAS import flow to detect if the user is trying to import a password-protected export, and provide an error alert if they do indicating we do not currently support that.
đ¸ 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