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

Submission result request handling fix (EXPOSUREAPP-3001) #1291

Merged
merged 2 commits into from
Oct 2, 2020

Conversation

BMItr
Copy link
Contributor

@BMItr BMItr commented Oct 2, 2020

Relates to

https://jira.itc.sap.com/browse/EXPOSUREAPP-3001

Description

If you provide a testresult e.g. by QR Code the result is usually pending until the final result is there.
In case of a final Positiv Result the background requests (fetchTestResults) are stopped (this is expected behaviour).
In case of a final Negativ Result there was an ongoing resultFetching by App-Resuming in Background even the final result there.
TestResult will only be fetched in correct states now.

@BMItr BMItr requested a review from a team October 2, 2020 09:53
Copy link
Contributor

@kolyaopahle kolyaopahle left a comment

Choose a reason for hiding this comment

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

passed local testing, LGTM 👍

@sonarcloud
Copy link

sonarcloud bot commented Oct 2, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@harambasicluka harambasicluka changed the title submission result request handling fix (EXPOSUREAPP-3001) Submission result request handling fix (EXPOSUREAPP-3001) Oct 2, 2020
@harambasicluka harambasicluka added 1.5.0 maintainers Tag pull requests created by maintainers labels Oct 2, 2020
@BMItr BMItr merged commit a07897d into release/1.5.x Oct 2, 2020
@BMItr BMItr deleted the fix/3001-submission-request-handling-fix branch October 2, 2020 10:58
@harambasicluka harambasicluka added this to the 1.5.0 milestone Oct 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
maintainers Tag pull requests created by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants