-
-
Notifications
You must be signed in to change notification settings - Fork 491
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(unstorage): add ttl support #970
feat(unstorage): add ttl support #970
Conversation
Someone is attempting to deploy a commit to a Personal Account owned by @pilcrowonpaper on Vercel. @pilcrowonpaper first needs to authorize it. |
Do you know if the adapter breaks if you use this with |
@pilcrowonpaper I believe nothing would happen for older versions of unstorage and for drivers that don't support ttl. TransactionOptions were introduced here unjs/unstorage#168 (v1.2), so maybe you could set the minimum version to 1.2+ ? |
I guess this doesn't matter since since you can pass arbitrary arguments in Javascript without any errors. Can you add a new changeset with |
Sorry I'm a little confused, this project isn't using changeset, what should I do ? Edit nvm, figured out the Auri stuff 👍🏽 |
Thanks! |
Ttl support is being added to unstorage through unjs/unstorage#262
For now the vercel-kv and redis drivers are supported, but as more support is added, this will start working for all of them.