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
Cookies fix #159
Cookies fix #159
Conversation
OlegMoshkovich
commented
Mar 22, 2022
- fix get cookies
- fix privacy slider
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there was a new test to add too?
800442b
to
a552a73
Compare
|
@pablo-mayrgundter PTAL! |
Please include link to your demo in the description |
@pablo-mayrgundter PTAL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please add tests for each exported method and then all looks good
src/privacy/analytics.js
Outdated
*/ | ||
export function isAnalyticsAllowed() { | ||
return Functional.getCookieBoolean('isAnalyticsAllowed', false) // defaultValue | ||
return Privacy.isPrivacySocialEnabled() // defaultValue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove comment
fix getCookie fix is Analytics allowed debuggin analytics - subtract isAllowed consitional add the debug built add privacy test refactor privacy function out of the about control space clean up refactor is analytics allowed method
5c3eada
to
0204019
Compare
@pablo-mayrgundter |
* fix privacy slider fix getCookie fix is Analytics allowed debuggin analytics - subtract isAllowed consitional add the debug built add privacy test refactor privacy function out of the about control space clean up refactor is analytics allowed method * new build * add is privacy social and usage enabled * additional test for privacy * clean up * new build