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

Fix app crash when closing cert details after notification (EXPOSUREAPP-14542) #5794

Merged
merged 8 commits into from
Jan 11, 2023

Conversation

SamuraiKek
Copy link
Contributor

@SamuraiKek SamuraiKek commented Jan 10, 2023

Ticket.

How to test:

  • Scan a certificate: cwa vc gen -e mock-cloud
  • Revoke the signature: cwa dcc rl -e mock-cloud --data -i -> paste the barcode data of the certificate you just scanned, hit enter and select "Revoke Signature" and hit enter again.
  • In the app, go to the test menu -> DCC State Validation, and refresh the revocation list.
  • When you get a notification, tap on it to go to the certificate details screen.
  • Close the screen and return to the certificates screen. No crash should take place anymore.

@SamuraiKek SamuraiKek added bug Something isn't working maintainers Tag pull requests created by maintainers labels Jan 10, 2023
@SamuraiKek SamuraiKek added this to the 3.1.x milestone Jan 10, 2023
@SamuraiKek SamuraiKek requested a review from a team January 10, 2023 11:17
@schauersbergern schauersbergern self-assigned this Jan 11, 2023
@mtwalli mtwalli merged commit 9a49997 into release/3.1.x Jan 11, 2023
@mtwalli mtwalli deleted the fix/14542-certificates-app-crash branch January 11, 2023 15:34
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

No Coverage information No Coverage information
No Duplication information No Duplication information

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants