-
Notifications
You must be signed in to change notification settings - Fork 97
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(IT Wallet): [SIW-1128] Add EidCard
and EidCardPreview
components
#5772
Conversation
EIdCard
and EIdCardPreview
components
EIdCard
and EIdCardPreview
componentsEIdCard
and EIdCardPreview
components
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5772 +/- ##
==========================================
+ Coverage 48.42% 49.55% +1.13%
==========================================
Files 1488 1619 +131
Lines 31617 32140 +523
Branches 7669 7724 +55
==========================================
+ Hits 15311 15928 +617
+ Misses 16238 16158 -80
+ Partials 68 54 -14
... and 497 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just checked the design guidelines provided by our colleagues and everything seems fine
EIdCard
and EIdCardPreview
componentsEidCard
and EidCardPreview
components
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
> [!WARNING] > This PR depends on #5779, #5772 ## Short description This PR adds the eID preview screen with mocked data. > [!important] > The screen is missing the component that allows scrolling to the bottom of the page. There is currently an incompatibility with the `IOScrollViewWithLargeHeader` component which will be resolved in a separate PR ## List of changes proposed in this pull request - Added `ItwIssuanceEidPreviewScreen` component - Added required locales - Added mock data for eID, DC, DL and TS in `ItwMockUtils` ## How to test Navigate to **Profile > IT Wallet > eID > Select an IDP**. You should be able to see the new screen as the below preview. ## Preview https://github.com/pagopa/io-app/assets/6160324/9d30201f-9c16-4593-8c24-9dab38377389 --------- Co-authored-by: Damiano Plebani <damiano.plebani@pagopa.it> Co-authored-by: Mario Perrotta <mario.perrotta@pagopa.it>
Short description
This PR introduces the
EidCard
andEidCardPreview
component for the IT Wallet EId credentialList of changes proposed in this pull request
EidCard
componentEidCardPreview
componentHow to test
Go to Profile > Design System > Cards and scroll to the bottom of the screen. Check that the components are rendered correctly
Preview
RocketSim_Recording_iPhone_14_6.1_2024-05-16_22.02.17.mp4