Basic Layout for Test Certificate Detail Screen (DEV) #3300
Basic Layout for Test Certificate Detail Screen (DEV) #3300
Conversation
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.
Small comments
.../de/rki/coronawarnapp/greencertificate/ui/certificates/details/CertificateDetailsFragment.kt
Outdated
Show resolved
Hide resolved
.../de/rki/coronawarnapp/greencertificate/ui/certificates/details/CertificateDetailsFragment.kt
Outdated
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_greencertificate_details.xml
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_greencertificate_details.xml
Outdated
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_greencertificate_details.xml
Outdated
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_greencertificate_details.xml
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_greencertificate_details.xml
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/greencertificate_qrcode_card.xml
Outdated
Show resolved
Hide resolved
...i/coronawarnapp/greencertificate/ui/certificates/details/GreenCertificateDetailsViewModel.kt
Outdated
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_greencertificate_details.xml
Outdated
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_greencertificate_details.xml
Outdated
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_greencertificate_details.xml
Outdated
Show resolved
Hide resolved
Corona-Warn-App/src/main/res/layout/fragment_greencertificate_details.xml
Outdated
Show resolved
Hide resolved
@axelherbstreith @harambasicluka can it be that there's one data input field missing? I didn't find the "Date and Time of Sample Collection | Datum und Uhrzeit der Probenahme" field. |
Nice Catch! This is somehow missing in the Android Figma file. Will clarify with the designers 👍 |
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.
As a basic PR for further development - LGTM 🚀
But please remember to address comments from @mtwalli
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.
Tested on S8 👍
This reverts commit bca7c56.
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 🚀
…//github.com/corona-warn-app/cwa-app-android into feature/dev-test-certificate-detail-fragment
ed2efb0
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.
UA approved
Kudos, SonarCloud Quality Gate passed! |
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
Basic Layout for GreenCertificate Detail Screen
To test, pls go to Test Menu -> Green Certificate -> Details Screen
Everything else will be done in a follow-up PR