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

Enable non-personalized Ads to support GDPR compliance #97

Merged
merged 3 commits into from
Apr 12, 2019

Conversation

carlos-algms
Copy link
Contributor

According to EU regulations, it is mandatory to enable users to opt-out to tracking cookies
GPT allows us to respect this law using the npa flag on Ads.

*  Enable non-personalized Ads

According to EU regulations, it is mandatory to enable users to opt-out to tracking cookies
GPT allows us to respect this law using the `npa` flag on Ads.

* Adding npa flag to re-render props

* Fix npa variable in componentWillReceiveProps
@CLAassistant
Copy link

CLAassistant commented Apr 4, 2019

CLA assistant check
All committers have signed the CLA.

@codecov
Copy link

codecov bot commented Apr 4, 2019

Codecov Report

Merging #97 into master will increase coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #97      +/-   ##
=======================================
+ Coverage   93.96%   94%   +0.04%     
=======================================
  Files           8     8              
  Lines         580   584       +4     
=======================================
+ Hits          545   549       +4     
  Misses         35    35
Impacted Files Coverage Δ
src/Bling.js 91.13% <100%> (+0.17%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 60f6ce0...63391a4. Read the comment docs.

@carlos-algms carlos-algms changed the title Enable non-personalized Ads Enable non-personalized Ads to support GDPR compliance Apr 5, 2019
@carlos-algms
Copy link
Contributor Author

This PR will fix #81

@carlos-algms
Copy link
Contributor Author

@miblanchard do you have any idea on when a new release will be launched?

@adityasundaram
Copy link

Any ETA on a release date for react-gpt with NPA flag?

@carlos-algms
Copy link
Contributor Author

@adityasundaram you can use https://www.npmjs.com/package/@carlos.algms/react-gpt while we wait for a new official release.

We are using it here, at my work, on our official system.

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.

5 participants