You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Maybe we can take this opportunity and rename this to `transaction_count_limit`.
I know it was merged like that in this PR, but this one will soon bring yet another similar name. Having such generic name as transactions_limit could be confusing.
Also, we might consider introducing a "limiter" abstraction in a follow-up ticket, instead of having three separate params.
I know it was merged like that in this PR, but this one will soon bring yet another similar name. Having such generic name as
transactions_limit
could be confusing.Also, we might consider introducing a "limiter" abstraction in a follow-up ticket, instead of having three separate params.
Originally posted by @rafal-ch in #2189 (comment)
The text was updated successfully, but these errors were encountered: