corona-warn-app / cwa-website

Corona-Warn-App website. The CWA development ended on May 31, 2023. You still can warn other users until April 30, 2023. More information:
https://coronawarn.app/en/faq/#ramp_down
Apache License 2.0
522 stars 225 forks source link

Certificate screenshots missing details #1804

Closed MikeMcC399 closed 1 year ago

MikeMcC399 commented 3 years ago

Avoid duplicates

Technical details

Describe the bug

Certificate screenshots shown on https://www.coronawarn.app/en/screenshots/, for instance Android Vaccination Certificate are missing details of the certificate. Only the top and the bottom of the scrolled certificate details are shown. The middle section is missing.

image

The files are:


Compare to the equivalent iOS Vaccination Certificate. The screenshot on the right is showing details which are not shown for Android.

image

Steps to reproduce the issue

  1. In Android Studio run VaccinationDetailsFragmentTest.kt on Pixel 3a emulator
  2. View > Tools Windows > Device File Explorer
  3. Go to /data/data/de.rki.coronawarnapp.test/screenshots
  4. View screenshots

Expected behavior

Certificate screenshots should show all details of a certificate, not just the top and bottom of a scrolled certificate.

Possible Fix

Scroll through the certificate view and capture each screen, not just the top and bottom of the scrolled view.

Ein-Tim commented 2 years ago

The real problem here is that the screenshots are out of date with version 2.20. They show a QR code on the detail screen of the person, although the QR code has been removed from this screen with version 2.20.

Suggestion how to proceed: Close this issue and open a new one addressing the out of date screenshots. See also my comment in https://github.com/corona-warn-app/cwa-website/issues/1760#issuecomment-1100198898.

MikeMcC399 commented 2 years ago

@Ein-Tim

There are multiple issues with screenshots. The point you raise about them being out of date is valid, however the subject of this issue is about the missing details in the Android vaccination certificate on https://www.coronawarn.app/en/screenshots/#android_vaccination_certificate is still true, so I do not see any reason to close this issue. You could consider it an enhancement request.

2.23 Android images from website:

image

The right-hand screenshot is of little interest.