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
521 stars 224 forks source link

TAN procedure FAQ #3063

Closed MikeMcC399 closed 2 years ago

MikeMcC399 commented 2 years ago

Where to find the issue

Describe the issue

There are multiple issues and inconsistencies with the TAN Procedure FAQ added through PR https://github.com/corona-warn-app/cwa-website/pull/3060 on July 29, 2022.

Inconsistencies with other usage:

  1. The virus name is written inconsistently as "Sars-CoV-2", differing from accepted capitalization.
  2. The hotline telephone numbers are formatted as 0800-7540002 and +49.30.49875402 differing from renderings of the phone number in other places in the FAQ and in the footer of the website.
  3. The main page of the app is referred to as "the Home Screen" differing from almost all other FAQ articles which refer to the "Status tab" instead.
  4. The English instructions contain the wrong text "Are you getting tested?" / "Next" / "Your PCR test was positive?" which are literal translations of the German app text, and not the text which is actually used by the app.
  5. The app name is inconsistently rendered as "Corona Warn App" (missing hyphens) and "Corona alert app" in the English text.

Other points:

Suggestion

  1. Write the virus as "SARS-CoV-2" to be consistent with other places on the website and with RKI: COVID-19 (Coronavirus SARS-CoV-2).
  2. Write the hotline telephone number as 0800 7540002 and +49 30 498 75402.
  3. Use the term "Status" to refer to the app "Home Screen".
  4. Correct the tile texts to read "Manage Your Tests" / "Continue" / "Was your PCR test positive?".
  5. Correct typos and punctuation.
  6. Improve English translation.
  7. Refer to app consistently as either Corona-Warn-App or CWA.
MikeMcC399 commented 2 years ago

I will submit a PR to resolve this issue next week.

Ein-Tim commented 2 years ago

Hi @MikeMcC399,

thanks for this issue! Could you please add to the list (and fix with your PR) that the entry includes the following sentence: "To reasonably record encounters through contact tracing and alert these contacts, the Corona alert app must be installed on the smartphone for at least a period of time; the TAN hotline will verify that period.", which uses Corona alert app instead of Corona-Warn-App.

Thanks for opening this issue, I support your request to leave PRs open for a longer period of time, I only reviewed the PR content-wise, but I have to admit that at least some of the issues you now spotted & the wrong translation "Corona alert app" should have been spotted by me in my review. Sorry for missing them!

MikeMcC399 commented 2 years ago

@Ein-Tim

Good catch for the app name. I will add that to the list of issues and correct it.

Edit: Added to OP.

MikeMcC399 commented 2 years ago
christianneu commented 2 years ago

As already mentioned in PR #3067, the fix looks good so we can merge it. Therefore I would close the issue. Thanks for the changes.