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

DCC export filter and sorting (EXPOSUREAPP-13311) #5266

Merged
merged 3 commits into from
Jun 8, 2022

Conversation

chiljamgossow
Copy link
Contributor

@chiljamgossow chiljamgossow commented Jun 7, 2022

  • unit test

@chiljamgossow chiljamgossow added the maintainers Tag pull requests created by maintainers label Jun 7, 2022
@chiljamgossow chiljamgossow requested a review from a team June 7, 2022 16:10
@chiljamgossow chiljamgossow added this to the 2.24.x milestone Jun 7, 2022
@github-actions
Copy link

github-actions bot commented Jun 7, 2022

Unit Test Results - Device Release

   556 files  +1     556 suites  +1   6m 29s ⏱️ +14s
2 737 tests +1  2 737 ✔️ +1  0 💤 ±0  0 ±0 
2 786 runs  +1  2 786 ✔️ +1  0 💤 ±0  0 ±0 

Results for commit 98dfe60. ± Comparison against base commit 4de49fb.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jun 7, 2022

Unit Test Results - Device For Tester Release

   559 files  +1     559 suites  +1   6m 19s ⏱️ +26s
2 743 tests +1  2 743 ✔️ +1  0 💤 ±0  0 ±0 
2 792 runs  +1  2 792 ✔️ +1  0 💤 ±0  0 ±0 

Results for commit 98dfe60. ± Comparison against base commit 4de49fb.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jun 7, 2022

Firebase Test Results

    2 files  ±0      2 suites  ±0   5m 1s ⏱️ + 1m 3s
129 tests ±0  129 ✔️ ±0  0 💤 ±0  0 ±0 
258 runs  ±0  258 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 98dfe60. ± Comparison against base commit 4de49fb.

@sonarcloud
Copy link

sonarcloud bot commented Jun 7, 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

82.0% 82.0% Coverage
0.0% 0.0% Duplication

@mtwalli mtwalli self-assigned this Jun 8, 2022
@mtwalli mtwalli merged commit 20965a7 into feature/13018-export-all-certs Jun 8, 2022
@mtwalli mtwalli deleted the feature/13311-export-filter branch June 8, 2022 06:15
mtwalli added a commit that referenced this pull request Jun 8, 2022
…5250)

* Add ToDo

* Implement new Toolbar in person overview (EXPOSUREAPP-13260) (#5253)

* Toolbar icons

* Connect icons

* Tooltip

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

Co-authored-by: Alex Paulescu <alex.paulescu@gmail.com>
Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Adapt release not strings. (#5262)

* All certificates PDF export info screen (EXPOSUREAPP-13261) (#5258)

* Add pdf export info screen.

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Use new texts.

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* DCC export filter and sorting (EXPOSUREAPP-13311) (#5266)

* filter

* filter

* test

* Certs preview screen (EXPOSUREAPP-13262) (#5260)

* Toolbar icons

* Connect icons

* Tooltip

* Web Approach

* Inject data

* Lint

* Lint

* Common fields

* `use`

* rename

* Zoom

* unique template classes

* Adjust style

* Page

* Toolbar

* Remove `EU` flag

* toolbar actions

* Update CertificateTemplate.kt

* Print

* px

* Progress

* Size

* Enhancing

* Fix tests

* Avoid multi inflate

* Name

* `-`

* catch error

* Share PDF

* Update DccExportAllOverviewFragment.kt

* move to vm

* Refactoring

* Docs

* Log

* Progress / error messages

* Catch errors

* lint

* Alignment

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Format

* Main dispatcher

* create pdf

* Delete file

* Formatting

* Clean up

* lint

* Cleaning

* bind filteriing

* lint

* Avoid crash

Co-authored-by: Nikolaus Schauersberger <nikolaus.schauersberger@gmail.com>
Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Unify flag SVG paths (EXPOSUREAPP-13310) (#5267)

* Toolbar icons

* Connect icons

* Tooltip

* Web Approach

* Inject data

* Lint

* Lint

* Common fields

* `use`

* rename

* Zoom

* unique template classes

* Adjust style

* Page

* Toolbar

* Remove `EU` flag

* toolbar actions

* Update CertificateTemplate.kt

* Print

* px

* Progress

* Size

* Enhancing

* Fix tests

* Avoid multi inflate

* Name

* `-`

* catch error

* Share PDF

* Update DccExportAllOverviewFragment.kt

* move to vm

* Refactoring

* Docs

* Log

* Progress / error messages

* Catch errors

* lint

* Alignment

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Format

* Main dispatcher

* create pdf

* Delete file

* Formatting

* Clean up

* lint

* Cleaning

* bind filteriing

* lint

* Avoid crash

* unify flag svg paths

Co-authored-by: Mohamed Metwalli <mohamed.metwalli@sap.com>
Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

Co-authored-by: Alex Paulescu <alex.paulescu@gmail.com>
Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>
Co-authored-by: Chilja Gossow <49635654+chiljamgossow@users.noreply.github.com>
Co-authored-by: Nikolaus Schauersberger <nikolaus.schauersberger@gmail.com>
mtwalli added a commit that referenced this pull request Jun 9, 2022
* Add ToDo

* Toolbar icons

* Connect icons

* Tooltip

* Implement new Toolbar in person overview (EXPOSUREAPP-13260) (#5253)

* Toolbar icons

* Connect icons

* Tooltip

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

Co-authored-by: Alex Paulescu <alex.paulescu@gmail.com>
Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Web Approach

* Inject data

* Lint

* Lint

* Common fields

* `use`

* rename

* Zoom

* unique template classes

* Adjust style

* Page

* Toolbar

* Remove `EU` flag

* toolbar actions

* Update CertificateTemplate.kt

* Print

* px

* Progress

* Size

* Enhancing

* Fix tests

* Avoid multi inflate

* Name

* `-`

* catch error

* Share PDF

* Update DccExportAllOverviewFragment.kt

* Adapt release not strings. (#5262)

* move to vm

* Refactoring

* Docs

* Log

* All certificates PDF export info screen (EXPOSUREAPP-13261) (#5258)

* Add pdf export info screen.

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Use new texts.

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Progress / error messages

* Catch errors

* lint

* Alignment

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Update Corona-Warn-App/src/main/res/values-de/covid_certificate_strings.xml

Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>

* Format

* Main dispatcher

* create pdf

* Delete file

* Formatting

* Clean up

* lint

* DCC export filter and sorting (EXPOSUREAPP-13311) (#5266)

* filter

* filter

* test

* Cleaning

* bind filteriing

* lint

* Avoid crash

* Create CertificateTemplateTest.kt

* Lint

* Hide item by default (avoid glitch)

* Test injection

* Create DccExportAllOverviewViewModelTest.kt

* raw country code

* Update CertificateDataInjectorTest.kt

* Update DccExportAllOverviewViewModelTest.kt

Co-authored-by: Alex Paulescu <alex.paulescu@gmail.com>
Co-authored-by: KathrinLuecke <91470863+KathrinLuecke@users.noreply.github.com>
Co-authored-by: Nikolaus Schauersberger <nikolaus.schauersberger@gmail.com>
Co-authored-by: Chilja Gossow <49635654+chiljamgossow@users.noreply.github.com>
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.

2 participants