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

feat: store language code of the consent in the cookie #761

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

OndraM
Copy link
Contributor

@OndraM OndraM commented Dec 9, 2024

We use cookieconsent library with many languages on multi-lingual site. Because of that, is it important information for us to be able to know the language of the consent the user has given.

This is useful in callbacks (as we store the consent meta data including language also server-side) and also when reading the language code from the already given consent cookie.

This PR extends cookie data with languageCode property. The property is only stored in the cookie, and its use is completely up to library users - its not populated back to cookieconsent global state etc.

Copy link

vercel bot commented Dec 9, 2024

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

Name Status Preview Comments Updated (UTC)
cookieconsent-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 9, 2024 2:08pm

Copy link

netlify bot commented Dec 9, 2024

Deploy Preview for cookieconsentv3-playground canceled.

Name Link
🔨 Latest commit d5223e0
🔍 Latest deploy log https://app.netlify.com/sites/cookieconsentv3-playground/deploys/6756f9c6a51f360008a5d421

@orestbida
Copy link
Owner

LGTM, thanks for the contribution!

@orestbida orestbida merged commit 8ff02e1 into orestbida:master Dec 18, 2024
7 checks passed
@OndraM OndraM deleted the get-language-code branch December 19, 2024 09:41
@OndraM
Copy link
Contributor Author

OndraM commented Jan 2, 2025

LGTM, thanks for the contribution!

Thanks and happy new year to you! Any estimate when you plan next CookieConsent release?

@orestbida
Copy link
Owner

@OndraM sorry for the late reply, should be up by the end of the week!

@OndraM
Copy link
Contributor Author

OndraM commented Jan 23, 2025

@OndraM sorry for the late reply, should be up by the end of the week!

Great, we are waiting for this release :) . Thank you!

@OndraM
Copy link
Contributor Author

OndraM commented Feb 4, 2025

@orestbida Sorry for reminding this, I very well know you do this in your free time, but do you have any ETA? We are depending on the change and its release and its blocking us right now. 😞

Thanks for your work!

@orestbida
Copy link
Owner

sorry for the delay, had forgotten ...

done!

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.

2 participants