-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat: make legacy consent management fields source specific #1121
feat: make legacy consent management fields source specific #1121
Conversation
…m-fields-source-specific-sdk-998
…m-fields-source-specific-sdk-998
…m-fields-source-specific-sdk-998
…m-fields-source-specific-sdk-998
…m-fields-source-specific-sdk-998
This PR is considered to be stale. It has been open for 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
…m-fields-source-specific-sdk-998
This PR is considered to be stale. It has been open for 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
…m-fields-source-specific-sdk-998
This PR is considered to be stale. It has been open for 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
This PR is considered to be stale. It has been open for 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
This PR is considered to be stale. It has been open for 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
This PR is considered to be stale. It has been open for 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
This PR is considered to be stale. It has been open for 20 days with no further activity thus it is going to be closed in 7 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
Closing this in favour of #1651 |
Description of the change
As we plan to support generic consent management for client-side SDKs, to make the backward compatibility feasible, we must make the legacy fields source-specific instead of
defaultConfig
.This PR addresses it for all the destinations. Extended the Ketch consent management for all the destinations.
Also, test data has been created for both the consent management fields.
Apologies for the huge PR.
Other changes:
Checklists
Development
Code review