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

Enable export of all certificates of a CWA user (EXPOSUREAPP-13018) #5250

Merged
merged 12 commits into from
Jun 8, 2022

Conversation

mtwalli
Copy link
Contributor

@mtwalli mtwalli commented Jun 1, 2022

@mtwalli mtwalli added enhancement Improvement of an existing feature maintainers Tag pull requests created by maintainers labels Jun 1, 2022
@mtwalli mtwalli added this to the 2.24.x milestone Jun 1, 2022
@github-actions
Copy link

github-actions bot commented Jun 2, 2022

Unit Test Results - Device Release

   555 files  +1     555 suites  +1   6m 15s ⏱️ +18s
2 734 tests +1  2 734 ✔️ +1  0 💤 ±0  0 ±0 
2 783 runs  +1  2 783 ✔️ +1  0 💤 ±0  0 ±0 

Results for commit 1cb4410. ± Comparison against base commit 84241fe.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jun 2, 2022

Unit Test Results - Device For Tester Release

   558 files  +1     558 suites  +1   5m 59s ⏱️ -31s
2 740 tests +1  2 740 ✔️ +1  0 💤 ±0  0 ±0 
2 789 runs  +1  2 789 ✔️ +1  0 💤 ±0  0 ±0 

Results for commit 1cb4410. ± Comparison against base commit 84241fe.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jun 2, 2022

Firebase Test Results

    2 files  ±0      2 suites  ±0   3m 26s ⏱️ -50s
129 tests ±0  129 ✔️ ±0  0 💤 ±0  0 ±0 
258 runs  ±0  258 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 1cb4410. ± Comparison against base commit 84241fe.

♻️ This comment has been updated with latest results.

mtwalli and others added 10 commits June 2, 2022 09:45
* 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>
* 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>
* 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>
* 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>
@mtwalli mtwalli marked this pull request as ready for review June 8, 2022 11:25
@mtwalli mtwalli requested review from a team June 8, 2022 11:25
@chiljamgossow chiljamgossow self-assigned this Jun 8, 2022
@sonarqubecloud
Copy link

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

11.7% 11.7% Coverage
0.1% 0.1% Duplication

@mtwalli mtwalli merged commit 84d5d4d into release/2.24.x Jun 8, 2022
@mtwalli mtwalli deleted the feature/13018-export-all-certs branch June 8, 2022 12:22
Copy link
Contributor

@KathrinLuecke KathrinLuecke left a comment

Choose a reason for hiding this comment

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

UA approved

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Improvement of an existing feature maintainers Tag pull requests created by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants