bitwarden / ios

Bitwarden mobile app for iOS.
https://bitwarden.com
GNU General Public License v3.0
212 stars 26 forks source link

[PM-10403] Added SSH key cipher item view #921

Closed fedemkr closed 1 month ago

fedemkr commented 1 month ago

🎟ī¸ Tracking

PM-10403

📔 Objective

Add new SSH key cipher item type view and use it in the cipher item view and edition.

Note: This is not the complete feature as it needs the SSH key item to be added to the SDK to completely work. But added the new behavior in all places possible before the SDK update.

📸 Screenshots

View SSH Key cipher item

View SSH Key cipher item

Edit SSH Key cipher item

Edit SSH Key cipher item

⏰ Reminders before review

đŸĻŽ Reviewer guidelines

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 97.57282% with 5 lines in your changes missing coverage. Please review.

Project coverage is 89.10%. Comparing base (5fc811a) to head (ca2164b). Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...red/Core/Vault/Extensions/BitwardenSdk+Vault.swift 33.33% 2 Missing :warning:
...red/UI/Vault/VaultItem/ViewItem/ViewItemView.swift 94.59% 2 Missing :warning:
...t/VaultItem/AddEditItem/AddEditItemProcessor.swift 90.90% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #921 +/- ## ========================================== + Coverage 89.05% 89.10% +0.04% ========================================== Files 654 656 +2 Lines 41057 41261 +204 ========================================== + Hits 36562 36764 +202 - Misses 4495 4497 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 1 month ago

Logo Checkmarx One – Scan Summary & Details – 60d86ec0-5307-4673-86fb-5876e2f36903

No New Or Fixed Issues Found