-
Notifications
You must be signed in to change notification settings - Fork 624
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: denom traces order #4709
fix: denom traces order #4709
Conversation
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.
Thanks, @emidev98!
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.
thank you @emidev98!
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.
Thank you!
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.
The order actually has implications on grpc web? interesting!
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.
That's what we learned back in time, truly it was mind blowing for me too, but somehow makes sense that the order affects the result
…point when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go
…point when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go
…point when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go
…point when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go
…point when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go
…point when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go
…point when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go
* fix: order query service rpcs to fix availability of denom traces endpoint when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go * add changelog * fix conflicts --------- Co-authored-by: Emi <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]>
* fix: order query service rpcs to fix availability of denom traces endpoint when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go * add changelog * fix conflicts --------- Co-authored-by: Emi <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]>
* fix: order query service rpcs to fix availability of denom traces endpoint when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go * add changelog * fix conflicts * fix some markdown links * fix more markdown links * fix even more links --------- Co-authored-by: Emi <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]>
* fix: order query service rpcs to fix availability of denom traces endpoint when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go * add changelog * fix conflicts * fix markdown links --------- Co-authored-by: Emi <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]>
* fix: order query service rpcs to fix availability of denom traces endpoint when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go * fix conflicts * add changelog --------- Co-authored-by: Emi <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]>
* fix: order query service rpcs to fix availability of denom traces endpoint when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go * add changelog * fix conflicts --------- Co-authored-by: Emi <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]>
* fix: order query service rpcs to fix availability of denom traces endpoint when no args are provided (#4709) Co-authored-by: Carlos Rodriguez <[email protected]> Co-authored-by: Jim Fasarakis-Hilliard <[email protected]> (cherry picked from commit 1283efb) # Conflicts: # modules/apps/transfer/types/query.pb.go # modules/apps/transfer/types/query.pb.gw.go * add changelog * fix conflicts --------- Co-authored-by: Emi <[email protected]> Co-authored-by: Carlos Rodriguez <[email protected]>
Description
Changes the order of DenomTraces and DenomTrace rpc services to fix an issue when querying the endpoint using LCD. The query defaults to
GET:/ibc/apps/transfer/v1/denom_traces/{hash=**}
instead of/ibc/apps/transfer/v1/denom_traces
when no parameter is available because of the RPC order definition.An example of the issue:
A similar issue has been fixed using the same approach on terra-money/alliance#54
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
).godoc
comments.Files changed
in the Github PR explorer.Codecov Report
in the comment section below once CI passes.