-
Notifications
You must be signed in to change notification settings - Fork 266
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
Replace cookie-universal-nuxt with a more generic library #11321
Comments
At the current state, I just found out that this library is set to keep cookies data updated into the global. |
@cnotv and I had an out of band discussion on this. The existing approach in #11354 should satisfy our need to remove nuxt dependencies, but we might want to follow-up in a separate PR to address using the inject pattern with cookies. The implementation in #11354 has been tested and confirmed to be working in Vue3, so a larger refactor away from the inject pattern shouldn't be required for the near-term. |
I rephrase this issue to match to actual final implementation. |
Description
Some logic has also been removed from a removal attempt of
cookie-universal-nuxt
(#11272).As this creates several issues, we want to replace it with
cookie-universal
(found by @rak-phillip).Logic replacement to be discussed/handled with #11392
The text was updated successfully, but these errors were encountered: