Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

[INFO]: Wrong "Aktualisiert: ..." / "Updated: ..." - Information on CWA start screen #1024

Closed
Jo-Achim opened this issue Aug 15, 2020 · 13 comments
Assignees
Labels
bug Something isn't working further input needed Issue requires more input from the creator to be processed good first issue mirrored-to-jira This item is also tracked internally in JIRA

Comments

@Jo-Achim
Copy link

Jo-Achim commented Aug 15, 2020

Android 10 with Android Sicherheitsupdate / Security-Update: 1. Juli 2020
Nokia 7.1, Build: 00WW_4_15C_SP02
Internet: WLAN (only)
CWA-Version: 1.2.1

Hello,

to see the little problem, do...

  1. Start CWA.
  2. If CWA starts with "Prüfung läuft ..." (CWA-Start-Screen) then look at the message "Aktualisiert: ..." ("Updated: ...").
    You will see: "Aktualisiert: Gestern ..." ("Updated: Yesterday ..."). / In my case: "Low risk".
  3. Touch the "->" to go to the "Ihr Risikostatus" / "Your Riskstatus" screen.
  4. Here you will see the correct: "Aktualisiert: Heute ..." / "Updated: Today ...".
  5. Tapp on "X" to go back to the start screen again and NOW the correct message "Aktualisiert: Heute ..." / "Updated: Today ..." was displayed.

Instead of executing point 3, it is sufficient, for example, to switch off the smartphone / restart the CWA app.

The indication of the wrong information is repeatable.

Best regards.


Internal Tracking ID: EXPOSUREAPP-2216

@Jo-Achim Jo-Achim added the bug Something isn't working label Aug 15, 2020
@vaubaehn
Copy link
Contributor

vaubaehn commented Aug 15, 2020

Same here. CWA1.2.1, ENF 16203019000 (ENF updated between yesterday and today)
But only on first start of CWA today before scheduled automatic update.
Not able to reproduce behaviour after switching off phone and restarting CWA, like suggested above.

@Jo-Achim
Copy link
Author

Jo-Achim commented Aug 16, 2020

Hi @vaubaehn,

Not able to reproduce behaviour after switching off phone and restarting CWA, like suggested above.

Follow steps 1 and 2.
3a. Then, instead to open then ‘Screenshot-Menu’ press the button too short, so the phone goes to ‘stand-by’.
3b. Restart it (e.g. per fingerprint-scanner) and the last CWA-Start-Screen is displayed.
After this, yesterday I saw the correct "Aktualisiert: Heute ..." / "Updated: Today ..." – instead of the wrong 'Gestern ...' / Yesterday ...' display under 2.

But today: after restarting the phone (3b.) the “Prüfung läuft …” is running and then: "URSACHE: 9002, Etwas ist schiefgelaufen. Timeout #998"!
{After some tapps to repeat and make screenshots, I see a 'time-out 60000 seconds' - We can probably put this into the box "a lot of bad actions in a short time by user" ;-) - well, not so important.}
But after this the correct “Aktualisiert: Heute ..." / "Updated: Today ..." was displayed.
So I have actually no additional information.

I'm adding an informal cross-reference in the thread: "URSACHE: 9002, Etwas ist schiefgelaufen. Timeout #998".

@ghost ghost assigned JoachimFritsch Aug 17, 2020
@ghost ghost added the mirrored-to-jira This item is also tracked internally in JIRA label Aug 17, 2020
@Jo-Achim
Copy link
Author

Brief impression of today ...

CWA-Version: 1.2.1

1. Start CWA.

2. If CWA starts with "Prüfung läuft ..." (CWA-Start-Screen) then look at the message "Aktualisiert: ..." ("Updated: ...").
   You will see: "Aktualisiert: **Gestern** ..." ("Updated: **Yesterday** ..."). / In my case: "Low risk".

3. Touch the "->" to go to the "Ihr Risikostatus" / "Your Riskstatus" screen.

4. Here you will see the correct: "Aktualisiert: **Heute** ..." / "Updated: **Today** ...".

5. Tapp on "X" to go back to the start screen again and NOW the correct message "Aktualisiert: **Heute** ..." / "Updated: **Today** ..." was displayed.

Is it possible that the incorrect previous update information ("Aktualisiert: Gestern ..." / "Updated: Yesterday ...") is still displayed in step 4?
And, in this case, only with next step 5 then both on the 'CWA-Start-Screen' and then (after "->" again) on the "Ihr Risikostatus" / "Your risk status" screen the correct update time ("Aktualisiert: Heute ..." / "Updated: Today" ... ") is shown?

Best regards.

@Jo-Achim
Copy link
Author

Jo-Achim commented Sep 13, 2020

Brief impression of today ...

CWA-Version: 1.2.1

1. Start CWA.

2. If CWA starts with "Prüfung läuft ..." (CWA-Start-Screen) then look at the message "Aktualisiert: ..." ("Updated: ...").
   You will see: "Aktualisiert: **Gestern** ..." ("Updated: **Yesterday** ..."). / In my case: "Low risk".

3. Touch the "->" to go to the "Ihr Risikostatus" / "Your Riskstatus" screen.

4. Here you will see the correct: "Aktualisiert: **Heute** ..." / "Updated: **Today** ...".

5. Tapp on "X" to go back to the start screen again and NOW the correct message "Aktualisiert: **Heute** ..." / "Updated: **Today** ..." was displayed.

Is it possible that the incorrect previous update information ("Aktualisiert: Gestern ..." / "Updated: Yesterday ...") is still displayed in step 4?
And, in this case, only with next step 5 then both on the 'CWA-Start-Screen' and then (after "->" again) on the "Ihr Risikostatus" / "Your risk status" screen the correct update time ("Aktualisiert: Heute ..." / "Updated: Today" ... ") is shown?

Best regards.

Yes, it is possible - CWA 1.3.0.
And it is possible that both displays ('CWA Start Screen' and 'Your Risk Status') will immediately show the correct update time (low accu (dark mode)).

@ghost ghost assigned harambasicluka Sep 28, 2020
@daimpi
Copy link

daimpi commented Oct 25, 2020

Related to: #1081

@d4rken
Copy link
Member

d4rken commented Oct 30, 2020

The OPs description sounds like there is an issue with the updated data being passed around via LiveData classes. They are not optimal for this approach and we've addressed quite a few of those issues for upcoming updates

Especially pausing+resuming the UI and the recreation of some UI parts would fit that behavior. I think this behavior could already improve as a side-effect of other changes in 1.6, as we moved a lot of non-UI code from LiveData (suitable for the UI layer) to a more reactive approach using Kotlin Flow (suitable for internal layers), removing the need for UI elements to "manually call refresh" and using patterns where the UI is "notified".

We're checking if this can still be reproduced on newer builds to evaluate whether extra changes are needed, or this will be fixed as a side effected from refactoring for upcoming features.

Example change (though not necessary the one that will fix this):
https://github.com/corona-warn-app/cwa-app-android/blob/release/1.6.x/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/tracing/GeneralTracingStatus.kt#L22

@Jo-Achim
Copy link
Author

Jo-Achim commented Nov 12, 2020

Android 10 with Android Security Update: September 1st, 2020
Nokia 7.1, Build: 00WW_4_16B
CWA version: 1.6.0
ENF version: 18204215000

The problem described in my first post is under CWA 1.6.0. occurred again today.

@Jo-Achim
Copy link
Author

Android 10 with Android Security Update: September 1st, 2020
Nokia 7.1, Build: 00WW_4_16B
CWA version: 1.6.0
ENF version: 18204215000

Hello, here some screenshots from described problem (after the start of CWA and the 'Update...' triggered in this way)...

After CWA-start and 'Update...' (Mainscreen):
Screenshot_20201116-021437

After tap to [->]:
Screenshot_20201116-021447

Back to Mainscreen (tapped to [X]):
Screenshot_20201116-021458

"Covid-19" - Info...
Screenshot_20201116-021529

Best regards.

@dsarkar
Copy link
Member

dsarkar commented Dec 19, 2020

Dear @Jo-Achim, can you reproduce this issue with CWA 1.9.1. Feedback very much appreciated. Best regards, DS.


Corona-Warn-App Open Source Team

@dsarkar dsarkar added the further input needed Issue requires more input from the creator to be processed good first issue label Dec 19, 2020
@Jo-Achim
Copy link
Author

No, not at the moment - but I will pay attention again.

@dsarkar
Copy link
Member

dsarkar commented Dec 19, 2020

@Jo-Achim, thanks for the feedback. We monitor this some more days before, hopefully, we can close this issue as fixed. Bes, DS


Corona-Warn-App Open Source Team

@Jo-Achim
Copy link
Author

Jo-Achim commented Dec 21, 2020

@dsarkar: Hello, I think that after the update, which also led to this change...

Screenshot_20201221-190241_t

the problem of the 'different update dates / times' has been resolved. So I think this issue can be closed.
Many thanks.

Best regards, Joachim.

PS:
Android 10 with Android security update: November 1st, 2020
Nokia 7.1, Build: 00WW_4_16B_SP01
CWA version: 1.9.1
ENF: 18204713000

@dsarkar
Copy link
Member

dsarkar commented Dec 21, 2020

@Jo-Achim, and community, thanks for the feedback and contributions. We close this issue now. Best, DS


Corona-Warn-App Open Source Team

@dsarkar dsarkar closed this as completed Dec 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working further input needed Issue requires more input from the creator to be processed good first issue mirrored-to-jira This item is also tracked internally in JIRA
Projects
None yet
Development

No branches or pull requests

7 participants