-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
fix(community): bump @upstash/ratelimit
version
#6832
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Skipped Deployment
|
Flagging that this will break existing users for people using |
@@ -148,7 +148,7 @@ class UpstashRatelimitHandler extends BaseCallbackHandler { | |||
_name?: string | |||
): Promise<void> { | |||
if (this.tokenRatelimit) { | |||
const remaining = await this.tokenRatelimit.getRemaining(this.identifier); | |||
const { remaining } = await this.tokenRatelimit.getRemaining(this.identifier); |
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.
May be some way to make this backwards compatible and allow "^1.1.3 || ^2.0.3"
in deps above
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.
I changed the code so that it works in both versions.
I didn't understand how the package.json should be updated, can you explain a bit more?
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.
@CahidArda I just pushed up a commit allowing those versions
@upstash/ratelimit
version
Bumping Upstash Ratelimit version to benefit from the number of improvements we have made.