-
Notifications
You must be signed in to change notification settings - Fork 288
Feature/8356 warning on behalf/8980 request #3387
Feature/8356 warning on behalf/8980 request #3387
Conversation
… submission service to make it reuseable
…o ensure its up to date
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.
UA approved
@@ -53,6 +53,7 @@ final class ClientMock { | |||
|
|||
var onGetTestResult: ((String, Bool, TestResultHandler) -> Void)? | |||
var onSubmitCountries: ((_ payload: SubmissionPayload, _ isFake: Bool, _ completion: @escaping KeySubmissionResponse) -> Void) = { $2(.success(())) } | |||
var onSubmitOnBehalf: ((_ payload: SubmissionPayload, _ isFake: Bool, _ completion: @escaping KeySubmissionResponse) -> Void) = { $2(.success(())) } |
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.
sounds a bit strange :D
@@ -152,21 +152,19 @@ class ENAExposureSubmissionService: ExposureSubmissionService { | |||
with: self.symptomsOnset | |||
) | |||
|
|||
let unencryptedCheckinsEnabled = self.appConfigurationProvider.featureProvider.value(for: .unencryptedCheckinsEnabled) | |||
let unencryptedCheckinsEnabled = AppFeatureProvider(appConfig: appConfig).value(for: .unencryptedCheckinsEnabled) |
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.
👍
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 change would not take into account the settings from the dev-menu. Please see the decorater pattern implemented to add the dev-menu functionality. Maybe we should change the pattern. Because its not obvious.
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.
Ah, thanks for that hint! I've now reversed that change and instead made sure that self.appConfigurationProvider.featureProvider
is always up-to-date whenever the app config publisher comes back. Maybe it makes sense to get rid of the AppFeatureProvider(appConfig: appConfig)
again as the publisher is now more reliable?
…e, fall back to default app config
…up-to-date app config in publisher
…ng decorators stay in effect
Description
Introduces the
OnBehalfCheckinSubmissionService
and implements the actual submission of a checkin on behalf of an attendee that doesn't use the Corona-Warn-App.Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-8980