cookieconsent icon indicating copy to clipboard operation
cookieconsent copied to clipboard

Fix service reject issue & setCookieData() method deletes lastConsentTimestamp field

Open peaemaye opened this issue 1 year ago • 3 comments

Fixes #700

The retrieveState method of api.js currently sets the _enabledServices before _acceptedServices so the enabled services don't include services read from the cc-cookie. I think reversing this order is sufficient to fix it.

Fixes #702

The setCookieData() method can accidentally delete the lastConsentTimestamp field from the cc-cookie.

peaemaye avatar Jun 27 '24 09:06 peaemaye

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
cookieconsent-docs ⬜️ Ignored (Inspect) Visit Preview Jun 27, 2024 11:39am

vercel[bot] avatar Jun 27 '24 09:06 vercel[bot]

Deploy Preview for cookieconsentv3-playground canceled.

Name Link
Latest commit 8785e5050611fb8e6f2c9abb17f40ee2c3af119b
Latest deploy log https://app.netlify.com/sites/cookieconsentv3-playground/deploys/667d4f6d765e06000835d070

netlify[bot] avatar Jun 27 '24 09:06 netlify[bot]

I think this also fixes #696 which is the same as #700

peaemaye avatar Jun 28 '24 14:06 peaemaye

Came across the same issue of lastConsentTimestamp not being set (basically was getting deleted) and spent a lot of time debugging 🙈

danielpaul avatar Jul 02 '24 17:07 danielpaul

Thanks for your contribution!

orestbida avatar Jul 02 '24 18:07 orestbida