cwa-app-android icon indicating copy to clipboard operation
cwa-app-android copied to clipboard

false statement: rapid test no longer valid more than 21 days; test is only 14 days old

Open diablodale opened this issue 3 years ago • 3 comments

Main app UI writes false statement

Rapid Test
No longer Valid
Your test is more than 21 days old and...

Last test is listed in app Journal on 1 April 2022 -- therefore it is 14 days old. Yes, the test itself is invalid. The false count is 21.

Avoid duplicates

  • [X] Bug is not mentioned in the FAQ
  • Unknown, I have only Android phone.
  • [X] Bug is not already reported in another issue

Technical details

  • Device name: realme x50 pro 5g
  • Android version: 11
  • App version: 2.20.4

Steps to reproduce the issue

  1. Open corona warn app

Result

False statement on main UI. image

Click to go to Journal. Scroll down to newest test. Test was on 1 April. image

Unable to sent error report due to bug #5089

Expected behaviour

True statement: The test is invalid and is 14 days old.

diablodale avatar Apr 14 '22 17:04 diablodale

Did you ever change the time on your device while the test was registered?

Ein-Tim avatar Apr 14 '22 18:04 Ein-Tim

I did not. Checking the Android settings, the time and timezone are set automatically

Germany changed the time compared to UTC on 27 March 2022; but that was before the 1 April test.

diablodale avatar Apr 14 '22 18:04 diablodale

error report id: 0DEB64CA239A409A4AC5

diablodale avatar Apr 15 '22 16:04 diablodale

@diablodale Text has been updated in 2.26 https://github.com/corona-warn-app/cwa-app-android/pull/5377, does this fix the issue for you ?

mtwalli avatar Sep 07 '22 10:09 mtwalli

I can not verify any fix/changes at this time. I will be able to look again at this in November.

diablodale avatar Sep 08 '22 23:09 diablodale

@diablodale

I can not verify any fix/changes at this time. I will be able to look again at this in November.

The number of days is hardcoded as "2" into the text string in 2.26, for example:

https://github.com/corona-warn-app/cwa-app-android/blob/ef2b8c3905ce2200352b824df785d9e452b9aace/Corona-Warn-App/src/main/res/values/antigen_strings.xml#L38-L39

I suggest to close the issue based on code inspection.

MikeMcC399 avatar Sep 09 '22 05:09 MikeMcC399

@diablodale closing it for now, feel free to reopen if you can reproduce it

mtwalli avatar Sep 09 '22 05:09 mtwalli