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

UQS pop up is shown on top of other screens

Open Ein-Tim opened this issue 3 years ago • 23 comments

Avoid duplicates

  • [X] Bug is not mentioned in the FAQ
  • [X] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
  • [X] Bug is not already reported in another issue

Technical details

  • Device name: Huawei P Smart 2019
  • Android version: 10
  • App version: 2.13.3

Describe the bug

The UQS pop up is shown on top of other screens, here are some examples:

Steps to reproduce the issue

  1. Delete and reinstall the app
  2. Do not dismiss the pop up
  3. Create a QR code on https://www.coronawarn.app/en/eventregistration/ and scan it with an external QR code scanner
  4. Enter the app following the link
  5. See the issue which is shown in the first screenshot above
  6. Try the same with RA test result QR codes

Expected behaviour

The pop up should not be shown if the app is not displaying the normal screens ("Status", "Certificates", "Check-in" or "Jounal").


Internal Tracking ID: EXPOSUREAPP-10623

Ein-Tim avatar Nov 16 '21 23:11 Ein-Tim

Good morning @Ein-Tim, thanks for reporting: Internal Tracking ID: EXPOSUREAPP-10623

dsarkar avatar Nov 17 '21 07:11 dsarkar

Thanks for reporting, Tim!

fynngodau avatar Nov 17 '21 08:11 fynngodau

FYI PR https://github.com/corona-warn-app/cwa-app-android/pull/4607

dsarkar avatar Dec 29 '21 15:12 dsarkar

@Ein-Tim

Did you look at this again?

Regarding your steps to reproduce: with the 2.17.2 version I did not see the pop-up appear after resetting the app, only after reinstalling.

MikeMcC399 avatar Feb 06 '22 11:02 MikeMcC399

@MikeMcC399

I currently don't have access to the Android device anymore, thus I can't check, sorry.

Maybe @fynngodau would like to test this again?

Ein-Tim avatar Feb 06 '22 11:02 Ein-Tim

@Ein-Tim I tried on 2.18.1 and again, the pop-up does not appear after resetting the app. I simulated a re-install by clearing the app's data and I then scanned an event code. This took me to the onboarding screen "LET'S GET STARTED". After completing onboarding, the QR Code Scanner pop-up was shown on top of the Check In screen.

This means that the issue is not resolved.

It may be that this is considered acceptable behaviour though. Normally the pop-up would be shown once only after installation. It would then be dismissed by the user and never shown again.

MikeMcC399 avatar Mar 16 '22 11:03 MikeMcC399

  • I suggested in the new issue https://github.com/corona-warn-app/cwa-app-android/issues/4940 to remove the UQS pop-up. I don't think that this pop-up is needed any more. It was useful, when version 2.12 was released, since previously there were different places to scan QR codes depending on their contents. The Universal QR code scanner allows all CWA-related QR codes to be scanned from one button. Version 2.12 was released in October 2021 and by this time users should be accustomed to the functionality and should therefore not need a special one-time pop-up to remind them.

MikeMcC399 avatar Mar 16 '22 13:03 MikeMcC399

@dsarkar

Either this issue should be closed (and a new issue opened to cover the remaining issue) or the label https://github.com/corona-warn-app/cwa-app-android/labels/Fix%202%2E16 should be removed.

MikeMcC399 avatar May 04 '22 09:05 MikeMcC399

@Ein-Tim see comment https://github.com/corona-warn-app/cwa-app-android/issues/4400#issuecomment-1117104473. Following @MikeMcC399's suggestion, do you want to close this issue since the remaining issue is covered in https://github.com/corona-warn-app/cwa-app-android/issues/4940?

dsarkar avatar May 04 '22 09:05 dsarkar

This issue was never fixed, especially problematic is when the pop up is shown on top of screens although the UQS icon is not visible.

I'm unsure if I should close this issue although it was not addressed. I think it should stay open until either @MikeMcC399's enhancement request us implemented or the bug fixed.

Ein-Tim avatar May 04 '22 09:05 Ein-Tim

@Ein-Tim

If you want to leave this issue open, then I suggest you update the steps to reproduce, because in current versions, resetting the app does not cause the pop-up to re-appear.

MikeMcC399 avatar May 04 '22 10:05 MikeMcC399

Done @MikeMcC399

Ein-Tim avatar May 04 '22 10:05 Ein-Tim

@Ein-Tim

Thanks for updating the steps to reproduce. The app is unusable unless the "QR Code Scanner" pop-up is dismissed, so it is quite intrusive. Dismissing the pop-up is however quite a simple operation (tap on "X").

If this issue is to stay open, then the labels https://github.com/corona-warn-app/cwa-app-android/labels/Fix%202%2E16 and https://github.com/corona-warn-app/cwa-app-android/labels/ready-to-close should be removed.

MikeMcC399 avatar May 04 '22 10:05 MikeMcC399

@dsarkar Please remove the labels as @MikeMcC399 suggested above.

Ein-Tim avatar Jun 05 '22 16:06 Ein-Tim

I did re-open the closed Jira ticket EXPOSUREAPP-10623

svengabr avatar Jun 08 '22 14:06 svengabr

FYI PR https://github.com/corona-warn-app/cwa-app-android/pull/5366

dsarkar avatar Jul 20 '22 11:07 dsarkar

@dsarkar Are you sure that this PR prevents the showing of the UQS tool tip on top of other screens? I'm only seeing changes in regard to the tap area in the PR.

Ein-Tim avatar Jul 20 '22 11:07 Ein-Tim

@Ein-Tim You are right. PR is still interesting here since a related topic.

dsarkar avatar Jul 20 '22 11:07 dsarkar

@dsarkar / @Ein-Tim

I have commented in https://github.com/corona-warn-app/cwa-app-android/pull/5366#issuecomment-1190193726 and suggested delaying the display of the pop-up. That would also fit in with stopping the pop-up displaying on screens where no scan button is displayed.

MikeMcC399 avatar Jul 20 '22 12:07 MikeMcC399

@Ein-Tim

  • PR https://github.com/corona-warn-app/cwa-app-android/pull/5366 has been changed to stop the UQS tooltip appearing on a new install, so your steps to reproduce in the OP will no longer show the issue.

MikeMcC399 avatar Jul 21 '22 11:07 MikeMcC399

@Ein-Tim

  • You might like to try out release/2.26.x since the reworked PR #5366 has been merged. Perhaps your expectations are now met in order to resolve this issue?

MikeMcC399 avatar Jul 22 '22 15:07 MikeMcC399

@MikeMcC399

I currently don't use Android Studio, but as I wanted to begin with it anyways, I will probably take this chance and take a look at it as soon as I have time.

Ein-Tim avatar Jul 22 '22 15:07 Ein-Tim

@Ein-Tim I checked it this in 2.26 and can't reproduce, probably because of the changes made by https://github.com/corona-warn-app/cwa-app-android/pull/5366.

mtwalli avatar Aug 01 '22 09:08 mtwalli

I currently don't have access to my Android device, but as soon as I get back home I'll test if this bug is fixed successfully with version 2.26.

Ein-Tim avatar Aug 31 '22 19:08 Ein-Tim

@Ein-Tim

  • If you delete and re-install the app, then no UQS pop-up is shown with CWA Android 2.26.
  • I already closed the related issue https://github.com/corona-warn-app/cwa-app-android/issues/4940.

MikeMcC399 avatar Sep 03 '22 12:09 MikeMcC399

@MikeMcC399 Okay, thanks! This issue seems fixed so I'm closing it now.

Ein-Tim avatar Sep 03 '22 21:09 Ein-Tim