Closed ttl33 closed 4 days ago
[!WARNING]
Review failed
The pull request is closed.
The recent changes in proto/dydxprotocol/vault/query.proto
are focused on streamlining the HTTP paths used for querying vault information in the Query
service. This simplification involves removing redundant segments from the paths, which makes the API endpoints more concise and easier to use.
File | Change Summary |
---|---|
proto/dydxprotocol/vault/query.proto | Simplified HTTP paths for querying a specific vault (/dydxprotocol/vault/{type}/{number} ) and all vaults (/dydxprotocol/vault ). |
In vaults where data sleeps so tight,
A change has come to paths, quite light.
Redundant bits have bid goodbye,
Simplified routes now amplify.
With ease we query, less to thread,
In dydxprotocol, we forge ahead! 😊
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Changelist
Remove extra
/vault
in query pathTest Plan
n/a
Author/Reviewer Checklist
state-breaking
label.indexer-postgres-breaking
label.PrepareProposal
orProcessProposal
, manually add the labelproposal-breaking
.feature:[feature-name]
.backport/[branch-name]
.refactor
,chore
,bug
.Summary by CodeRabbit