-
Notifications
You must be signed in to change notification settings - Fork 40
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
Cannot remove Falsy cookies #58
Comments
@Heziode can I see the code you've used for setting up the cookie? I think you may be right though, I'll check the code when I have the time, thanks for reporting this. |
Example from my project where I encountered the problem: Example of setting (here is a boolean value): Example of deletion: I have some cookies that can have a "falsy" value depending of user preferences. |
Hi @Heziode, I have updated the cookie library with the fix. Can you please close this issue if it works? |
Thank's @microcipcip, it works. |
I trying to remove "falsy" cookies (cookies that contains
0
, orfalse
), but I cannot.I think this is due to:
cookie-universal/packages/cookie-universal/index.js
Line 89 in a703d5d
Current workaround: set a non "falsy" value to be removable.
The text was updated successfully, but these errors were encountered: