IBC rate limit query

[grarco]

Describe your changes

Adds a query to retrieve the ibc rate limit for a specific token.

Checklist before merging

• If this PR has some consensus breaking changes, I added the corresponding breaking:: labels
  • This will require 2 reviewers to approve the changes
• If this PR requires changes to the docs or specs, a corresponding PR is opened in the namada-docs repo
  • Relevant PR if applies:
• If this PR affects services such as namada-indexer or namada-masp-indexer, a corresponding PR is opened in that repo
  • Relevant PR if applies:

[grarco]

@brentstone I've also added a cli command for the query. Let me know if we only need the sdk function so that I can revert that change

[codecov]

Codecov Report

Attention: Patch coverage is 43.39623% with 30 lines in your changes missing coverage. Please review.

Project coverage is 74.02%. Comparing base (c70475b) to head (3859e51).
Report is 12 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/sdk/src/rpc.rs	24.32%	28 Missing ⚠️
crates/ibc/src/parameters.rs	50.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4246      +/-   ##
==========================================
- Coverage   74.03%   74.02%   -0.01%     
==========================================
  Files         345      345              
  Lines      109877   109920      +43     
==========================================
+ Hits        81349    81370      +21     
- Misses      28528    28550      +22     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[brentstone]

@grarco I've fixed up some things and I think the PR is rdy now

[grarco]

Actually, given the way the ibc vp validates transactions, this logic is wrong. The sdk query_ibc_rate_limits function returns an error if just one of the two keys is missing and in here we fallback to the default values for both, but the vp does actually support setting only one of the two and then using the default param for the other one. I'll try to rework this

[grarco]

Done in 9a1b9b3

[brentstone]

thx, gotcha!

[tzemanovic]

just 1 suggestion, otherwise lgtm