-
Notifications
You must be signed in to change notification settings - Fork 288
Fix/6551 error messages and other bugs #2488
Conversation
…cwa-app-ios into fix/6551-error-messages * 'fix/6551-error-messages' of github.com:corona-warn-app/cwa-app-ios: missing variable placeholder change of text renaming issues change in folder changes based on feedback add logic adding strings adding tests
…cwa-app-ios into fix/6551-error-messages * 'fix/6551-error-messages' of github.com:corona-warn-app/cwa-app-ios: Remove some unneeded newlines Add a comment on why Rearrange Backgroundtask reset coordinators when showing onboarding pass route to home after onboarding
src/xcode/ENA/ENA/Source/Scenes/Home/Cells/TestResult/HomeTestResultCellModel.swift
Show resolved
Hide resolved
src/xcode/ENA/ENA/Source/Scenes/Home/Cells/TestResult/HomeTestResultCellModel.swift
Show resolved
Hide resolved
self?.configureLoading() | ||
onUpdate() | ||
if self?.coronaTestService.pcrTest?.finalTestResultReceivedDate == nil { | ||
if testResultIsLoading { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isTestResultLoading ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This way it makes a correct english sentence: "If test result is loading [do x]" I admit being inconsistent with this, but would like to keep it that way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's more of a personal style and how you prefer, actually I saw isActivityIndicatorHidden in the next line, so in that context, I found it to be little inconsistent. Anyhow, you have my vote :)
Description
Fixes several bugs:
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-6525
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-6526
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-6527
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-6550
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-6551
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-6571