From 4a30f45ca6f3c375be8ffd119612c924fb7f6d6e Mon Sep 17 00:00:00 2001 From: Felix Schmidt Date: Tue, 14 Mar 2023 12:08:27 +0100 Subject: [PATCH 1/5] 14916: Adopt Thank You! Screen and show a tappable FAQ link. --- .../Localization/bg.lproj/Localizable.links.strings | 2 +- .../Localization/de.lproj/Localizable.links.strings | 2 +- .../Resources/Localization/de.lproj/Localizable.strings | 4 ++-- .../Localization/en.lproj/Localizable.links.strings | 2 +- .../Localization/pl.lproj/Localizable.links.strings | 2 +- .../Localization/ro.lproj/Localizable.links.strings | 2 +- .../Localization/tr.lproj/Localizable.links.strings | 2 +- .../Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.swift | 1 + .../Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.xib | 9 ++++++--- .../EndOfLifeThankYouCellViewModel.swift | 9 ++++----- 10 files changed, 19 insertions(+), 16 deletions(-) diff --git a/src/xcode/ENA/ENA/Resources/Localization/bg.lproj/Localizable.links.strings b/src/xcode/ENA/ENA/Resources/Localization/bg.lproj/Localizable.links.strings index 5196e364eb0..dcc244d738f 100644 --- a/src/xcode/ENA/ENA/Resources/Localization/bg.lproj/Localizable.links.strings +++ b/src/xcode/ENA/ENA/Resources/Localization/bg.lproj/Localizable.links.strings @@ -68,4 +68,4 @@ "ExposureSubmission_WarnWithoutTAN_FAQLink" = "https://www.coronawarn.app/en/faq/results/#warn_without_tan"; -"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/de/faq/"; // to.do final URL +"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/en/faq/#ramp_down"; diff --git a/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.links.strings b/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.links.strings index 4928b237f3c..61b2cb739d0 100644 --- a/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.links.strings +++ b/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.links.strings @@ -70,4 +70,4 @@ "ExposureSubmission_WarnWithoutTAN_FAQLink" = "https://www.coronawarn.app/de/faq/results/#warn_without_tan"; -"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/de/faq/"; // to.do final URL +"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/de/faq/#ramp_down"; diff --git a/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.strings b/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.strings index 841a6b18794..4fd1a2795a7 100644 --- a/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.strings +++ b/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.strings @@ -1525,9 +1525,9 @@ Bei ausgeschalteter Hintergrundaktualisierung müssen Sie die App täglich aufru "Home_EndOfLifeThankYouTile_Title" = "Vielen Dank!"; -"Home_EndOfLifeThankYouTile_description" = "Vielen Dank für die Nutzung der Corona-Warn-App. Während der Pandemie wurden durch die Nutzenden der Corona-Warn-App rund 9 Millionen positive Testergebnisse geteilt. Durch diese Teilung positiver Testergebnisse konnten Personen schnell vor einem erhöhten Infektionsrisiko gewarnt werden, das ihnen sonst unerkannt geblieben wäre. Viele Gewarnte wurden anschließend positiv getestet. Die Corona-Warn-App hat dank Ihrer Mithilfe ihren Zweck der frühzeitigen Warnung erfüllt und die Beendigung von Infektionsketten unterstützt.\n\nAb dem 1. Juni 2023 wird die Corona-Warn-App nicht mehr weiterentwickelt. Sie können weiterhin auf Ihre bereits in der CWA gespeicherten Zertifikate und das Kontakt-Tagebuch zugreifen. Alle anderen Funktionen stehen nicht mehr zur Verfügung.\n\nWeitere Informationen zum Betriebsende finden Sie in den %@."; +"Home_EndOfLifeThankYouTile_description" = "Vielen Dank für die Nutzung der Corona-Warn-App. Während der Pandemie wurden durch die Nutzenden der Corona-Warn-App rund 9 Millionen positive Testergebnisse geteilt. Durch diese Teilung positiver Testergebnisse konnten Personen schnell vor einem erhöhten Infektionsrisiko gewarnt werden, das ihnen sonst unerkannt geblieben wäre. Viele Gewarnte wurden anschließend positiv getestet. Die Corona-Warn-App hat dank Ihrer Mithilfe ihren Zweck der frühzeitigen Warnung erfüllt und die Beendigung von Infektionsketten unterstützt.\n\nAb dem 1. Juni 2023 wird die Corona-Warn-App nicht mehr weiterentwickelt. Sie können weiterhin auf Ihre bereits in der CWA gespeicherten Zertifikate und das Kontakt-Tagebuch zugreifen. Alle anderen Funktionen stehen nicht mehr zur Verfügung.\n\n%@"; -"Home_EndOfLifeThankYouTile_faq" = "FAQ"; +"Home_EndOfLifeThankYouTile_faq" = "Mehr Informationen finden Sie in den FAQ."; /* Home Test Registration Card */ diff --git a/src/xcode/ENA/ENA/Resources/Localization/en.lproj/Localizable.links.strings b/src/xcode/ENA/ENA/Resources/Localization/en.lproj/Localizable.links.strings index 6f344b57ee8..ba5dc036119 100644 --- a/src/xcode/ENA/ENA/Resources/Localization/en.lproj/Localizable.links.strings +++ b/src/xcode/ENA/ENA/Resources/Localization/en.lproj/Localizable.links.strings @@ -70,4 +70,4 @@ "ExposureSubmission_WarnWithoutTAN_FAQLink" = "https://www.coronawarn.app/en/faq/results/#warn_without_tan"; -"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/de/faq/"; // to.do final URL +"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/en/faq/#ramp_down"; diff --git a/src/xcode/ENA/ENA/Resources/Localization/pl.lproj/Localizable.links.strings b/src/xcode/ENA/ENA/Resources/Localization/pl.lproj/Localizable.links.strings index 06ef125f466..909dde4de12 100644 --- a/src/xcode/ENA/ENA/Resources/Localization/pl.lproj/Localizable.links.strings +++ b/src/xcode/ENA/ENA/Resources/Localization/pl.lproj/Localizable.links.strings @@ -68,4 +68,4 @@ "ExposureSubmission_WarnWithoutTAN_FAQLink" = "https://www.coronawarn.app/en/faq/results/#warn_without_tan"; -"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/de/faq/"; // to.do final URL +"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/en/faq/#ramp_down"; diff --git a/src/xcode/ENA/ENA/Resources/Localization/ro.lproj/Localizable.links.strings b/src/xcode/ENA/ENA/Resources/Localization/ro.lproj/Localizable.links.strings index 593c53f2f34..0b4101174f4 100644 --- a/src/xcode/ENA/ENA/Resources/Localization/ro.lproj/Localizable.links.strings +++ b/src/xcode/ENA/ENA/Resources/Localization/ro.lproj/Localizable.links.strings @@ -68,4 +68,4 @@ "ExposureSubmission_WarnWithoutTAN_FAQLink" = "https://www.coronawarn.app/en/faq/results/#warn_without_tan"; -"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/de/faq/"; // to.do final URL +"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/en/faq/#ramp_down"; diff --git a/src/xcode/ENA/ENA/Resources/Localization/tr.lproj/Localizable.links.strings b/src/xcode/ENA/ENA/Resources/Localization/tr.lproj/Localizable.links.strings index 7d07fc69362..e67b1b6bf99 100644 --- a/src/xcode/ENA/ENA/Resources/Localization/tr.lproj/Localizable.links.strings +++ b/src/xcode/ENA/ENA/Resources/Localization/tr.lproj/Localizable.links.strings @@ -68,4 +68,4 @@ "ExposureSubmission_WarnWithoutTAN_FAQLink" = "https://www.coronawarn.app/en/faq/results/#warn_without_tan"; -"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/de/faq/"; // to.do final URL +"Home_EndOfLifeThankYouTile_FAQLink" = "https://www.coronawarn.app/en/faq/#ramp_down"; diff --git a/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.swift b/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.swift index 6c5c1d28e0f..9b5779a6705 100644 --- a/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.swift +++ b/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.swift @@ -60,6 +60,7 @@ final class EndOfLifeThankYouCell: UITableViewCell { descriptionTextView.adjustsFontForContentSizeCategory = true descriptionTextView.backgroundColor = .clear descriptionTextView.delegate = self + descriptionTextView.linkTextAttributes = [NSAttributedString.Key.foregroundColor: UIColor.enaColor(for: .textTint)] setupAccessibility() } diff --git a/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.xib b/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.xib index 7beb50edea0..66c9cd08ec5 100644 --- a/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.xib +++ b/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCell.xib @@ -25,7 +25,7 @@ - + - + Lorem ipsum dolor sit er elit lamet, consectetaur cillium adipisicing pecu, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. @@ -94,6 +94,9 @@ + + + diff --git a/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCellViewModel.swift b/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCellViewModel.swift index 35df94dfc3c..3983c50d422 100644 --- a/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCellViewModel.swift +++ b/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCellViewModel.swift @@ -32,11 +32,10 @@ class EndOfLifeThankYouCellViewModel { attributes: textAttributes ) - // to.do 08.03.2023: wait for final decision whether the FAQ link is tappable or not - // attributedString.mark( - // faqLinkText, - // with: LinkHelper.urlString(suffix: faqLinkAnchor, type: .faq) - // ) + attributedString.mark( + faqLinkText, + with: faqLinkAnchor + ) return attributedString } From f575cc056b527d355ac968b1eaa4ec4e006e9e7c Mon Sep 17 00:00:00 2001 From: Felix Schmidt Date: Tue, 14 Mar 2023 12:08:57 +0100 Subject: [PATCH 2/5] 14916: Split the guard to get better log informations. --- .../Extensions/NSMutableAttributedString+Link.swift | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/src/xcode/ENA/ENA/Source/Extensions/NSMutableAttributedString+Link.swift b/src/xcode/ENA/ENA/Source/Extensions/NSMutableAttributedString+Link.swift index 19e4ff4743e..6437da55a37 100644 --- a/src/xcode/ENA/ENA/Source/Extensions/NSMutableAttributedString+Link.swift +++ b/src/xcode/ENA/ENA/Source/Extensions/NSMutableAttributedString+Link.swift @@ -9,9 +9,14 @@ extension NSMutableAttributedString { /// looks for the given text and sets a link attribute public func mark(_ text: String, with link: String) { let foundRange = mutableString.range(of: text) - guard let linkURL = URL(string: link), - foundRange.location != NSNotFound else { - Log.debug("Link \(text) text not found") + + guard let linkURL = URL(string: link) else { + Log.debug("Link URL could not created from string: \(link).") + return + } + + guard foundRange.location != NSNotFound else { + Log.debug("Link \(text) text not found.") return } From b504d44f6cbc5afe703b6c7639da6e6749b34fad Mon Sep 17 00:00:00 2001 From: Felix Schmidt Date: Tue, 14 Mar 2023 12:14:15 +0100 Subject: [PATCH 3/5] 16916: Replace the word CWA with App. --- .../ENA/ENA/Resources/Localization/de.lproj/Localizable.strings | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.strings b/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.strings index 4fd1a2795a7..837506b96b9 100644 --- a/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.strings +++ b/src/xcode/ENA/ENA/Resources/Localization/de.lproj/Localizable.strings @@ -1525,7 +1525,7 @@ Bei ausgeschalteter Hintergrundaktualisierung müssen Sie die App täglich aufru "Home_EndOfLifeThankYouTile_Title" = "Vielen Dank!"; -"Home_EndOfLifeThankYouTile_description" = "Vielen Dank für die Nutzung der Corona-Warn-App. Während der Pandemie wurden durch die Nutzenden der Corona-Warn-App rund 9 Millionen positive Testergebnisse geteilt. Durch diese Teilung positiver Testergebnisse konnten Personen schnell vor einem erhöhten Infektionsrisiko gewarnt werden, das ihnen sonst unerkannt geblieben wäre. Viele Gewarnte wurden anschließend positiv getestet. Die Corona-Warn-App hat dank Ihrer Mithilfe ihren Zweck der frühzeitigen Warnung erfüllt und die Beendigung von Infektionsketten unterstützt.\n\nAb dem 1. Juni 2023 wird die Corona-Warn-App nicht mehr weiterentwickelt. Sie können weiterhin auf Ihre bereits in der CWA gespeicherten Zertifikate und das Kontakt-Tagebuch zugreifen. Alle anderen Funktionen stehen nicht mehr zur Verfügung.\n\n%@"; +"Home_EndOfLifeThankYouTile_description" = "Vielen Dank für die Nutzung der Corona-Warn-App. Während der Pandemie wurden durch die Nutzenden der Corona-Warn-App rund 9 Millionen positive Testergebnisse geteilt. Durch diese Teilung positiver Testergebnisse konnten Personen schnell vor einem erhöhten Infektionsrisiko gewarnt werden, das ihnen sonst unerkannt geblieben wäre. Viele Gewarnte wurden anschließend positiv getestet. Die Corona-Warn-App hat dank Ihrer Mithilfe ihren Zweck der frühzeitigen Warnung erfüllt und die Beendigung von Infektionsketten unterstützt.\n\nAb dem 1. Juni 2023 wird die Corona-Warn-App nicht mehr weiterentwickelt. Sie können weiterhin auf Ihre bereits in der App gespeicherten Zertifikate und das Kontakt-Tagebuch zugreifen. Alle anderen Funktionen stehen nicht mehr zur Verfügung.\n\n%@"; "Home_EndOfLifeThankYouTile_faq" = "Mehr Informationen finden Sie in den FAQ."; From 2789fd7099c13900ce8a7808180326b498685725 Mon Sep 17 00:00:00 2001 From: Felix Schmidt Date: Wed, 15 Mar 2023 11:08:55 +0100 Subject: [PATCH 4/5] 14916: Swiftlint. --- .../HealthCertifiedPersonCellModel.swift | 1 + .../EndOfLifeThankYou/EndOfLifeThankYouCellViewModel.swift | 6 +----- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/src/xcode/ENA/ENA/Source/Scenes/HealthCertificates/Overview/Cells/HealthCertifiedPerson/HealthCertifiedPersonCellModel.swift b/src/xcode/ENA/ENA/Source/Scenes/HealthCertificates/Overview/Cells/HealthCertifiedPerson/HealthCertifiedPersonCellModel.swift index b42c0f0d7f0..adb12982221 100644 --- a/src/xcode/ENA/ENA/Source/Scenes/HealthCertificates/Overview/Cells/HealthCertifiedPerson/HealthCertifiedPersonCellModel.swift +++ b/src/xcode/ENA/ENA/Source/Scenes/HealthCertificates/Overview/Cells/HealthCertifiedPerson/HealthCertifiedPersonCellModel.swift @@ -10,6 +10,7 @@ class HealthCertifiedPersonCellModel { // MARK: - Init + // swiftlint:disable:next cyclomatic_complexity init?( healthCertifiedPerson: HealthCertifiedPerson, cclService: CCLServable, diff --git a/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCellViewModel.swift b/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCellViewModel.swift index 3983c50d422..410f6b30a7f 100644 --- a/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCellViewModel.swift +++ b/src/xcode/ENA/ENA/Source/Scenes/Home/Cells/EndOfLifeThankYou/EndOfLifeThankYouCellViewModel.swift @@ -14,7 +14,6 @@ class EndOfLifeThankYouCellViewModel { var description: NSAttributedString { let faqLinkText = AppStrings.Home.EndOfLifeThankYouTile.faqLinkText - let faqLinkAnchor = AppStrings.Home.EndOfLifeThankYouTile.faqLinkAnchor let string = String( format: AppStrings.Home.EndOfLifeThankYouTile.description, @@ -32,10 +31,7 @@ class EndOfLifeThankYouCellViewModel { attributes: textAttributes ) - attributedString.mark( - faqLinkText, - with: faqLinkAnchor - ) + attributedString.mark(faqLinkText, with: AppStrings.Home.EndOfLifeThankYouTile.faqLinkAnchor) return attributedString } From 42aaa5bb9045aec0ba59f79d3ff882bd7ced861f Mon Sep 17 00:00:00 2001 From: Felix Schmidt Date: Wed, 15 Mar 2023 13:24:42 +0100 Subject: [PATCH 5/5] 14916: Swiftlint. --- .../HealthCertifiedPerson/HealthCertifiedPersonCellModel.swift | 1 - 1 file changed, 1 deletion(-) diff --git a/src/xcode/ENA/ENA/Source/Scenes/HealthCertificates/Overview/Cells/HealthCertifiedPerson/HealthCertifiedPersonCellModel.swift b/src/xcode/ENA/ENA/Source/Scenes/HealthCertificates/Overview/Cells/HealthCertifiedPerson/HealthCertifiedPersonCellModel.swift index adb12982221..b42c0f0d7f0 100644 --- a/src/xcode/ENA/ENA/Source/Scenes/HealthCertificates/Overview/Cells/HealthCertifiedPerson/HealthCertifiedPersonCellModel.swift +++ b/src/xcode/ENA/ENA/Source/Scenes/HealthCertificates/Overview/Cells/HealthCertifiedPerson/HealthCertifiedPersonCellModel.swift @@ -10,7 +10,6 @@ class HealthCertifiedPersonCellModel { // MARK: - Init - // swiftlint:disable:next cyclomatic_complexity init?( healthCertifiedPerson: HealthCertifiedPerson, cclService: CCLServable,