Update CipherViewExtensions to use previous state to avoid clearing user set values

david-livefront commented 1 week ago

🎟️ Tracking

N/A

📔 Objective

This PR updates CipherViewExtensions to use previous state to avoid clearing user set values like visibility and password re-prompt.

📸 Screenshots

Before	After

⏰ Reminders before review

Contributor guidelines followed
All formatters and local linters executed and passed
Written new unit and / or integration tests where applicable
Used internationalization (i18n) for all UI strings
CI builds passed
Communicated to DevOps any deployment requirements
Updated any necessary documentation or informed the documentation team

🦮 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

github-actions[bot] commented 1 week ago

Checkmarx One – Scan Summary & Details – 0216349e-39f9-4554-9e12-519aca94b791

No New Or Fixed Issues Found

codecov[bot] commented 1 week ago

Welcome to Codecov :tada:

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered :open_umbrella:

david-livefront commented 1 week ago

Thanks @brian-livefront

bitwarden / android