Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
IBC Rate Limiting #556
IBC Rate Limiting #556
Changes from 67 commits
950a274
21089a8
f991e57
6e2a037
32c67d5
8a608c5
d5e4098
573b0e8
43b8bca
3e7b583
acc565c
bff325c
fab8893
1e35a51
6b2cb18
8adfb9f
ebcae0b
8b5a0b9
2937b8d
d3a4369
c2e26da
d3612ad
2b25f18
10ee62a
f45ad70
12a0dfe
0200f69
c83d069
275cbdb
334de7f
bbf7ec1
1397864
9df5cbf
a2ef933
0d3b8c5
a2efe4e
e176282
e7abc29
67da44b
352601f
65200dc
59cdbba
ec41e24
15364df
ad01251
6be7aed
d02ce19
9bb5ef1
510d29d
e00d9ce
4213325
6ae4dd8
30e6e73
79a1930
02cfd23
93bd465
7d60f82
79d653e
c4f2618
3a69e45
560f7e2
1f3d75f
eb795bd
5409792
0d1c62e
4f46c2e
e4c1e9c
703d432
1a1bd4e
d3064f6
aa0af49
4fa3ad7
72b07ce
62d5857
fa505fc
8b5b394
e866d8e
dad4ec9
7bee669
90349c3
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
Should we implement these here and just leave them commented out?
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.
It'd require adding the whole upgrade handler, so I think I'd lean toward doing it in a separate PR. Wdyt?
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.
Agreed, makes sense to put in a separate PR. We should just make sure that goes in the next upgrade.