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

[CHEF-6102 CHEF-6114] GTM and OneTrust integration #3187

Merged
merged 6 commits into from Dec 8, 2023

Conversation

ayush-billore
Copy link
Contributor

@ayush-billore ayush-billore commented Nov 15, 2023

Description

  • Add GTM changes
  • Add Onetrust changes
  • Add feature flag for onetrust and GTM

https://chefio.atlassian.net/browse/CHEF-6102
https://chefio.atlassian.net/browse/CHEF-6114

Demo Video
https://github.com/chef/supermarket/assets/102516633/a3c385b2-f265-486f-9826-c8f2ddbe8e3d

Issues Resolved

[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]

Check List

@ayush-billore ayush-billore force-pushed the gtm-and-onetrust branch 2 times, most recently from bacd905 to 8b663c6 Compare November 22, 2023 07:17
@ayush-billore ayush-billore force-pushed the gtm-and-onetrust branch 5 times, most recently from 0cabe44 to 980b818 Compare December 4, 2023 14:22
@ayush-billore ayush-billore marked this pull request as ready for review December 4, 2023 14:23
@ayush-billore ayush-billore requested review from a team as code owners December 4, 2023 14:23
Signed-off-by: ayush-billore <ayush.billore@progress.com>
Signed-off-by: ayush-billore <ayush.billore@progress.com>
Signed-off-by: ayush-billore <ayush.billore@progress.com>
Signed-off-by: ayush-billore <ayush.billore@progress.com>
Signed-off-by: ayush-billore <ayush.billore@progress.com>
Signed-off-by: ayush-billore <ayush.billore@progress.com>
Copy link

sonarcloud bot commented Dec 8, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug B 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@ayush-billore ayush-billore marked this pull request as ready for review December 8, 2023 13:34
@saghoshprogress saghoshprogress merged commit def941f into main Dec 8, 2023
16 of 18 checks passed
@saghoshprogress saghoshprogress deleted the gtm-and-onetrust branch December 8, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants