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

Badge consistency (EXPOSUREAPP-13144) #5237

Merged
merged 11 commits into from
May 30, 2022
Merged

Conversation

chiljamgossow
Copy link
Contributor

Making sure badges are only counted if the corresponding tile is also visible

# Conflicts:
#	Corona-Warn-App/src/main/java/de/rki/coronawarnapp/covidcertificate/vaccination/ui/details/VaccinationDetailsFragment.kt
@chiljamgossow chiljamgossow added the maintainers Tag pull requests created by maintainers label May 25, 2022
@chiljamgossow chiljamgossow added this to the 2.24.x milestone May 25, 2022
@chiljamgossow chiljamgossow requested review from a team May 25, 2022 14:38
@sonarcloud
Copy link

sonarcloud bot commented May 30, 2022

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

43.6% 43.6% Coverage
0.0% 0.0% Duplication

Copy link
Contributor

@mtwalli mtwalli left a comment

Choose a reason for hiding this comment

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

LGTM

@mtwalli mtwalli self-assigned this May 30, 2022
@SamuraiKek SamuraiKek self-assigned this May 30, 2022
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.

@SamuraiKek SamuraiKek merged commit 2d19b4b into release/2.24.x May 30, 2022
@SamuraiKek SamuraiKek deleted the fix/13144-badge branch May 30, 2022 11:43
chiljamgossow added a commit that referenced this pull request Jun 7, 2022
* enhancement

* check for inconsistencies

* revert

* add test

* refactor

* refactor

* detekt klint
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

3 participants