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: implement URL encoding and new cookieAttributes prop #25

Merged
merged 9 commits into from
Sep 25, 2023

Conversation

switchnollie
Copy link
Contributor

@switchnollie switchnollie commented Sep 15, 2023

  • Drops cookieDomain in favor of a more flexible prop cookieAttributes
  • URL encoding/decoding for setting and parsing the cookie
  • New e2e tests to test entire journeys

Pull Request Checklist

@pboeder
Copy link
Member

pboeder commented Sep 25, 2023

I really like that! That needs to be released as a new major version

@switchnollie switchnollie force-pushed the 22-add-expirationdate-as-property branch from 4ef0cc7 to 33bd890 Compare September 25, 2023 11:58
@itrich
Copy link
Member

itrich commented Sep 25, 2023

I really like that! That needs to be released as a new major version

Awesome! Please keep in mind to trigger a new FOSS evaluation before pushing the new bundled release to npm. 😄

@switchnollie switchnollie merged commit a68cd66 into main Sep 25, 2023
3 checks passed
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.

3 participants