search

corona-warn-app / cwa-app-ccl

Common Covid Logic (CCL) for Android and iOS
Apache License 2.0
7 stars 4 forks source link

Suggestions for FAQ Link from Booster text for DE-BNR-0110 #88

Closed MikeMcC399 closed 2 years ago

MikeMcC399 commented 2 years ago

Where to find the issue

https://github.com/corona-warn-app/cwa-app-ccl/blob/97d66df83eb232824efe39db880b661c0c1b9e83/lib/ccl/functions/getDccWalletInfo.js#L461 defines the faqAnchor vac_cert_booster which displays in the CWA app as "For further information, please see our FAQ page." and links to the website https://www.coronawarn.app/en/faq/#vac_cert_booster.

Describe the issues

  1. There is a website issue logged as https://github.com/corona-warn-app/cwa-website/issues/3073 which causes a mobile device not to display FAQ booster information when the link is clicked on.

  2. The anchor #vac_cert_booster no longer exists on the website. It was removed when the FAQs were reorganized and instead the anchor is redirected to #vac_cert according to https://github.com/corona-warn-app/cwa-website/blob/bc2485d70bab67b865a6aad08c69cfa3f3815bd0/src/data/faq_redirects.json#L9.

Steps to reproduce

Mobile device is connected via WLAN

  1. Tap Scan and scan in a 2/2 vaccination certificate
  2. Tap back-arrow
  3. Tap Certificates
  4. Tap right-arrow
  5. Tap "Note on Booster Vaccination"

DE-BNR-0110

  1. Tap on link at bottom of text to Rule DE-BNR-0110 "For further information, please see our FAQ page."
  2. Note that browser first goes to https://www.coronawarn.app/en/faq/#vac_cert_booster then to https://www.coronawarn.app/en/faq/results/#vac_cert. The display however shows only topic sections.

image

Expectation

When using the link on a mobile device useful information related to the booster notice should be shown.

Suggestions

The following are some alternatives:

  1. Change the anchor #vac_cert_booster on CCL to point to the existing article with anchor #vac_booster_basics. The article is What is a booster vaccination and how does it appear in the CWA? and it seems to fit the information needs for a reader of DE-BNR-0110 most closely.

  2. Write a new FAQ article related to DE-BNR-0110, give it a new anchor, publish it and change CCL accordingly.

  3. Remove the anchor #vac_cert_booster from CCL.

  4. Fix issue https://github.com/corona-warn-app/cwa-website/issues/3073. Make no changes in CCL.

EXPOSUREAPP-13805

mlenkeit commented 2 years ago

Thanks @MikeMcC399 for raising this, mirrored to Jira as EXPOSUREAPP-13805.

MikeMcC399 commented 2 years ago

@mlenkeit

Thank you for picking up the suggestion and putting it into PR #90! Is there a concrete plan at the moment for moving this into production? I can see v3.2.0 marked as pre-release.

mlenkeit commented 2 years ago

@MikeMcC399 it will most likely be released with CWA 2.26 towards the end of the month.

MikeMcC399 commented 2 years ago

I noticed that the FAQ https://www.coronawarn.app/en/faq/results/#vac_booster_basics specifies a 3 month wait for a 1st booster vaccination, which is in alignment with the BNR rules but no longer in alignment with the 21st STIKO recommendations. I have logged:

to clarify and make changes if necessary.

MikeMcC399 commented 2 years ago

This change is implemented and live in CCL 3.2.1 in use by CWA 2.26.