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

UQS badges (EXPOSUREAPP-9327) #4083

Merged
merged 16 commits into from
Sep 20, 2021
Merged

UQS badges (EXPOSUREAPP-9327) #4083

merged 16 commits into from
Sep 20, 2021

Conversation

mtwalli
Copy link
Contributor

@mtwalli mtwalli commented Sep 17, 2021

Please test and review after #4089, so Tests badges can be noticed

Addresses : EXPOSUREAPP-9327

  • Scan any Certificate from anywhere , a new badge should appear marking the new certificate
  • Scan any corona Test (PCR / RAT) from anywhere a new badge should appear on the Home 馃彔 icon when user end up on a screen different from Home .(Please note that this can verified when @AlexanderAlferov task is done)

@mtwalli mtwalli marked this pull request as ready for review September 17, 2021 14:42
@mtwalli mtwalli requested a review from a team September 17, 2021 14:42
@marcauberer marcauberer added the maintainers Tag pull requests created by maintainers label Sep 19, 2021
@marcauberer marcauberer added this to the 2.11.0 milestone Sep 19, 2021
@AlexanderAlferov AlexanderAlferov self-assigned this Sep 20, 2021
@sonarcloud
Copy link

sonarcloud bot commented Sep 20, 2021

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

12.9% 12.9% Coverage
4.5% 4.5% Duplication

@SamuraiKek SamuraiKek self-assigned this Sep 20, 2021
Copy link
Contributor

@SamuraiKek SamuraiKek left a comment

Choose a reason for hiding this comment

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

LGTM. Works nicely.

@mtwalli mtwalli merged commit d298f20 into release/2.11.x Sep 20, 2021
@mtwalli mtwalli deleted the fix/9327-badges-again branch September 20, 2021 13:24
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.

None yet

4 participants