minvws / nl-covid19-coronacheck-app-ios

European Union Public License 1.2
25 stars 11 forks source link

Vaccination details data is incomplete, inconsistent and incorrect #44

Closed marcokrikke closed 3 years ago

marcokrikke commented 3 years ago

Describe the bug, issue or concern

I've received two vaccinations via GGD and see them twice in the app (RIVM + GGD).

When comparing the vaccination details with coronacheck.nl, data seems to be missing or inconsistent in the iOS app. Also within the app, data is incosistent between the vaccination details and the International QR Code details.

To Reproduce

Steps to reproduce the behavior:

  1. Get fully vaccinated
  2. Check if vaccinations are listed when adding a vaccination certificate
  3. Check details of vaccinations
  4. Create a QR code and check if it says 2/2 doses
  5. Log in to https://coronacheck.nl/nl/print/
  6. Check if vaccinations are listed when creating a vaccination certificate for priting
  7. Check details of vaccinations, compare with (3)

Expected behavior

Screenshots

GGD

App: GGD iOS app

Website: GGD coronacheck.nl

RIVM

App: RIVM iOS app

Website: RIVM coronacheck.nl

QR in app

IMG_1661

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context

Moved from #40 to a separate issue, since #40 got closed.

Governance

ijansch commented 3 years ago

Thanks for the remarks.

The 'is this the last dose: no' will be changed in the next release, as this is confusing. The line is only relevant when you receive only one Pfizer (because you previously had covid), and this indicates that you won't need a second.

As for the missing data in RIVM: IT's not actually missing, RIVM gives the app an 'hpkcode' from https://hpkcode.nl. When generating the certificate, you'll see that the missing data is enhanced from that json structure. However to avoid confusion, the next release will also look up the hpkcode in all of the apps and site, to populate those fields.

As for the 'identifier': the one in the 'wat is er opgehaald' should be sticky for your particular vaccination, so the same regardless of when/where you load it. The one that ends up in the eventual QR (the dcc certificate) is generated every time you create a new DCC. That is indeed intended behaviour. Should you notice the identifier from RIVM change between requests, feel free to log a separate issue for that.