[PM-7162] Cipher Form - Item Details
đī¸ Tracking
đ Objective
Lay the initial ground for the the new add/edit cipher form that makes full use of the CL. Includes the first Item Details section that will be re-used for all cipher types.
Note for reviewers -- I recommend stepping through each commit to more easily understand the changes in more digestible chunks. I tried my best to make them descriptive and build upon each other without introducing too much many changes at once. Also, a significant chunk of the line changes are related to spec files and storybook.
đ¸ Screenshots
See storybook for interactive demonstrations.
| Add New Cipher | Edit Existing Cipher |
|---|---|
| Partial Edit Cipher | Clone Cipher |
|---|---|
â° Reminders before review
- Contributor guidelines followed
- All formatters and local linters executed and passed
- Written new unit and / or integration tests where applicable
- Protected functional changes with optionality (feature flags)
- Used internationalization (i18n) for all UI strings
- CI builds passed
- Communicated to DevOps any deployment requirements
- Updated any necessary documentation (Confluence, contributing docs) 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
Codecov Report
Attention: Patch coverage is 40.62500% with 171 lines in your changes missing coverage. Please review.
Project coverage is 29.60%. Comparing base (
9294a4c) to head (2594931).
Additional details and impacted files
@@ Coverage Diff @@
## main #9758 +/- ##
==========================================
+ Coverage 29.55% 29.60% +0.05%
==========================================
Files 2538 2548 +10
Lines 74221 74490 +269
Branches 13874 13932 +58
==========================================
+ Hits 21935 22056 +121
- Misses 50637 50775 +138
- Partials 1649 1659 +10
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@shane-melton Just an fyi - your 'clone cipher' screen shot is missing
Checkmarx One â Scan Summary & Details â 45d1be61-1132-4d68-85bd-12fa30c5d4e2