Skip to content
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

[CDAP-20726] - adds conditional cookie banner for notifying users for analytics #1069

Merged
merged 4 commits into from
Aug 1, 2023

Conversation

seanfeld
Copy link
Collaborator

@seanfeld seanfeld commented Jul 20, 2023

Cookie notification banner for analytics

Description

Added conditional cookie notification banner for google analytics usage. Config changes need to be made that contain the banner text and the link.

PR Type

  • Bug Fix
  • Feature
  • Build Fix
  • Testing
  • General Improvement
  • Cherry Pick

Links

Jira: 20726

Screenshots

Screenshot 2023-07-26 at 1 01 48 PM

&& {
background-color: #333333;
padding: 20px;
border-radius: 4px;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

alphabetize?

@dli357 dli357 added the build triggers github action label Jul 31, 2023
@seanfeld seanfeld merged commit 038abe9 into develop Aug 1, 2023
2 checks passed
@seanfeld seanfeld deleted the podesta/cookie-banner branch August 1, 2023 20:44
seanfeld added a commit that referenced this pull request Aug 1, 2023
Cherry pick #1069 into release 6.9.2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.9.2 build triggers github action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants