-
-
Notifications
You must be signed in to change notification settings - Fork 67
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/add useCookie composable #745
Conversation
Thank you for the PR, this one is a bit tricky because of the dependency to
|
Thank you, I will contribute after learn @vue-composable/axios.🤗 |
Hi, pikax I added a new repo for useCookie and I learned a lot. |
# Conflicts: # packages/axios/package.json # yarn.lock
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, sorry for the delay.
Made a few changes
It's ok. In china, these day are new year's day, best wish for you. |
* feat(composable): add a new composable * chore: update tests to use fake timers * refactor(use-cookie): add new repo for useCookie. * chore: few changes * chore: fix example on readme * chore: remove extra peerDependency * chore: build Co-authored-by: pikax <carlos@hypermob.co.uk>
Hi, pikax
I added a new composable that is useCookie, it used js-cookie library.
can you review this PR?