-
Notifications
You must be signed in to change notification settings - Fork 2k
chore(website): Tim.sara/transcend removal #24419
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
Conversation
removes transcend file reference
deletes transcend file
|
All contributors have signed the CLA ✍️ ✅ |
|
Hi @timsara331, can you sign the CLA please? Also I don't think we can remove TrustArc without having a replacement for users to change their cookie settings |
|
@thomasqueirozb just wanted to follow up and say that I have signed the CLA. As for the replacement, we are replacing the TrustArc script with Transcend. This script will be loaded via GTM. This can be tested via the GTM preview: https://tagmanager.google.com/#/container/accounts/9450256/containers/67447237/workspaces/7/tags Once approved, I will update the GTM container to deploy new the triggers and tags that load Transcend. |
|
recheck |
|
I have read the CLA Document and I hereby sign the CLA |
Summary
Vector configuration
How did you test this PR?
Change Type
Is this a breaking change?
Does this PR include user facing changes?
no-changeloglabel to this PR.References
Notes
@vectordotdev/vectorto reach out to us regarding this PR.pre-pushhook, please see this template.make fmtmake check-clippy(if there are failures it's possible some of them can be fixed withmake clippy-fix)make testgit merge origin masterandgit push.Cargo.lock), pleaserun
make build-licensesto regenerate the license inventory and commit the changes (if any). More details here.