-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DIA-2193 generate USPString client side #448
DIA-2193 generate USPString client side #448
Conversation
@@ -4,6 +4,7 @@ | |||
// | |||
// Created by Andre Herculano on 08.02.21. | |||
// | |||
// swiftlint:disable line_length |
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.
Have you checked if it is possible to remove this ignore-linter line and fix the issue itself? Usually, line length is easy to fix.
ConsentViewController/Classes/SourcePointClient/SourcepointClientCoordinator.swift
Outdated
Show resolved
Hide resolved
@andresilveirah I would expect to see a little bit more unit tests for this feature ✌️ |
@Nevazhnovu just bringing the convo from slack here: |
The SDKs use to rely on the uspstring returned by the endpoints. However, the uspstring is dependent on ccpa.applies and there are cases in which ccpa.applies changes and the SDK doesn't necessarily need to call the endpoints that return uspstring.
With this PR, the SDK is now generating the uspstring with the data it has locally. If ccpa.applies changes, even if the SDK doesn't receive a new uspstring, it'll now be able to reflect the correct uspstring by generating it on the fly.