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

Prettify svg templates for pdf export (EXPOSUREAPP-13635) #5420

Merged
merged 3 commits into from
Aug 5, 2022

Conversation

schauersbergern
Copy link
Contributor

Ticket

Testing:

Scan following EU and non EU certificates:

cwa tc gen -e wru -o --seed 10 --iss AT -s t.0.co=AT --png
cwa vc gen -e wru -o --seed 10 --iss AT -s v.0.co=AT --png
cwa rc gen -e wru -o --seed 10 --iss AT -s r.0.co=AT --png
cwa tc gen -e wru -o --seed 10 --png
cwa vc gen -e wru -o --seed 10 --png
cwa rc gen -e wru -o --seed 10 --png

  • Trigger batch export and check if text looks good now
  • Export Certificates and check if text looks good now
  • Export pdf per Mail and check if text looks good now

Tested in Emulator, Pixel 6 and Samsung A3 where text looked good now

@schauersbergern schauersbergern added bug Something isn't working maintainers Tag pull requests created by maintainers labels Aug 4, 2022
@schauersbergern schauersbergern added this to the 2.26.x milestone Aug 4, 2022
@schauersbergern schauersbergern requested a review from a team August 4, 2022 13:29
@schauersbergern schauersbergern changed the title Prettify svg templates for pdf export Prettify svg templates for pdf export (EXPOSUREAPP-13635) Aug 4, 2022
@github-actions
Copy link

github-actions bot commented Aug 4, 2022

Unit Test Results - Device For Tester Release

   563 files  ±0     563 suites  ±0   6m 3s ⏱️ -1s
2 777 tests ±0  2 777 ✔️ ±0  0 💤 ±0  0 ±0 
2 826 runs  ±0  2 826 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 340b151. ± Comparison against base commit 6b29ede.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

Unit Test Results - Device Release

   560 files  ±0     560 suites  ±0   6m 25s ⏱️ +58s
2 771 tests ±0  2 771 ✔️ ±0  0 💤 ±0  0 ±0 
2 820 runs  ±0  2 820 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 340b151. ± Comparison against base commit 6b29ede.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 4, 2022

Firebase Test Results

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

Results for commit 340b151. ± Comparison against base commit 6b29ede.

♻️ This comment has been updated with latest results.

@mtwalli mtwalli linked an issue Aug 4, 2022 that may be closed by this pull request
3 tasks
@MikeMcC399
Copy link
Contributor

I can still see white space between the hyphenated words and the hyphen.

image

@MikeMcC399 MikeMcC399 mentioned this pull request Aug 4, 2022
3 tasks
@schauersbergern
Copy link
Contributor Author

I can still see white space between the hyphenated words and the hyphen.

image

@MikeMcC399 hyphens are fixed in new recent commit

@MikeMcC399
Copy link
Contributor

@schauersbergern

hyphens are fixed in new recent commit.

I checked by exporting a German vaccination certificate and the hyphens now look fine. 👍🏻
That is the only file which I can check.

@SamuraiKek SamuraiKek self-assigned this Aug 5, 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.

Text looks good now.

@sonarcloud
Copy link

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

No Coverage information No Coverage information
No Duplication information No Duplication information

@schauersbergern schauersbergern merged commit 138fb87 into release/2.26.x Aug 5, 2022
@schauersbergern schauersbergern deleted the fix/13635_pdf_export_templates branch August 5, 2022 13:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working maintainers Tag pull requests created by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extra whitespace in EU printout
3 participants