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

Hide progress dialog on missing consent from both Google and App (EXPOSUREAPP-5320) #2443

Merged
merged 1 commit into from
Feb 23, 2021

Conversation

mtwalli
Copy link
Contributor

@mtwalli mtwalli commented Feb 23, 2021

Before the fix:
1- Open the App and proceed with QR code scanning
2- Disallow Google consent
3- Scan a positive test
4- Proceed to sharing
5- Click on Consent button and disallow our consent flag
6- return back and try to share the keys
7- the progress dialog will be shown forever

After:
Dialog will hide if consent has not been given

@mtwalli mtwalli added bug Something isn't working maintainers Tag pull requests created by maintainers prio PRs to review first. labels Feb 23, 2021
@mtwalli mtwalli added this to the 1.13.0 milestone Feb 23, 2021
@mtwalli mtwalli requested a review from a team February 23, 2021 13:43
@sonarcloud
Copy link

sonarcloud bot commented Feb 23, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

10.0% 10.0% Coverage
0.0% 0.0% Duplication

@jurajkusnier jurajkusnier self-assigned this Feb 23, 2021
Copy link
Contributor

@jurajkusnier jurajkusnier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Tested on Pixel (Android 11)

@ralfgehrer ralfgehrer self-assigned this Feb 23, 2021
Copy link
Contributor

@ralfgehrer ralfgehrer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Tested on Pixel (Android 10).

@ralfgehrer ralfgehrer merged commit f77ae20 into release/1.13.x Feb 23, 2021
@ralfgehrer ralfgehrer deleted the fix/5320-hide-progress-no-consent branch February 23, 2021 14:38
@harambasicluka harambasicluka modified the milestones: 1.13.0, 1.13.1 Feb 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working maintainers Tag pull requests created by maintainers prio PRs to review first.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants