-
Notifications
You must be signed in to change notification settings - Fork 31
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
feat: add new query to get a swap #1387
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1387 +/- ##
==========================================
- Coverage 69.58% 69.57% -0.01%
==========================================
Files 668 668
Lines 56208 56231 +23
==========================================
+ Hits 39111 39125 +14
- Misses 14830 14835 +5
- Partials 2267 2271 +4
|
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.
+1 for describing the swagger fix in the PR description.
It would be also nice to give a hint about the fix regarding the missing pagination handling.
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.
There is no single swap query cli. Please check it.
Thanks for finding missing cli. I've just added. |
* feat: add new query to get a swap * chore: update changelog * fix: add missing pagination param in cli client * chore: lint fix * chore: add grpc-query test * chore: add descriptive message for invalid QuerySwapRequest * chore: add CLI to query swap (cherry picked from commit 92ad675) # Conflicts: # client/docs/swagger-ui/swagger.yaml
* feat: add new query to get a swap (#1387) * feat: add new query to get a swap * chore: update changelog * fix: add missing pagination param in cli client * chore: lint fix * chore: add grpc-query test * chore: add descriptive message for invalid QuerySwapRequest * chore: add CLI to query swap (cherry picked from commit 92ad675) # Conflicts: # client/docs/swagger-ui/swagger.yaml * chore: fix conflict --------- Co-authored-by: jaeseung-bae <119839167+jaeseung-bae@users.noreply.github.com>
Description
client/docs/config.json
QuerySwapsRequest
CLIMotivation and context
How has this been tested?
Screenshots (if appropriate):
Checklist:
CHANGELOG.md
client/docs/swagger-ui/swagger.yaml