-
-
Notifications
You must be signed in to change notification settings - Fork 140
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/Osmosis Chain/Connector #237
Closed
chasevoorhees
wants to merge
35
commits into
hummingbot:development
from
pecuniafinance:development-merged
Closed
Changes from 32 commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
d2eda9c
Merged with dev branch. Need to handle broken osmo-math pkg still.
chasevoorhees b871ec7
Fix file perms.
chasevoorhees 7a8d9ce
Fix file perms.
chasevoorhees 27f6bd1
coded in swap calcs locally
chasevoorhees c727aa0
coded in swap calcs locally
chasevoorhees 0395d9a
made it cosmosish again
chasevoorhees f61f9dc
remove cosmospriceresponse, merged into priceresponse
chasevoorhees 27264b9
remove commented tests and jest-free debug file
chasevoorhees 6b29392
fix schemas again
chasevoorhees 29de6ac
add cosmos-specific ymls to swagger
chasevoorhees 3fc3956
add cosmos-specific ymls to swagger
chasevoorhees a457fa2
cosmos definitions and routes
nkhrs 802fd74
update swap fee logic
chasevoorhees 6c31117
remove unused cosmos endpoints from swagger
chasevoorhees 67cca44
managed to add swagger same-paths with unique models
chasevoorhees 455fd64
balance swap swap back trade quantities for testwallet balance
chasevoorhees 7bca960
updated base fee to use EIP1559 by default and osmosis config for it.…
chasevoorhees a5d7f8a
Merge pull request #238 from hummingbot/staging
rapcmia bff7843
Merge branch 'hummingbot:main' into development-merged
nkhrs 082d093
Merge branch 'development' into development-merged
chasevoorhees 3905409
Merge branch 'development' into development-merged
chasevoorhees 70c18f5
fixed yarn.lock bad paste
chasevoorhees 6b6a0ec
update axios import into jest
chasevoorhees 0186916
fixed gateway balance and some more logic around IEP1559
chasevoorhees 89b7337
updated tendermint client to 37 so all tests work on testnet again. s…
chasevoorhees 3eaffb1
Merge branch 'development' into development-merged
chasevoorhees f771fb7
Merge pull request #261 from hummingbot/development
nikspz 6923d45
Merge branch 'main' into staging
nikspz 74a9158
Merge pull request #258 from hummingbot/staging
nikspz f07e9c6
changes WIP - models are finished for SWAGGER DOCS updates though - h…
chasevoorhees 2748c09
add test CURLs and json requests for osmosis
nkhrs 29ebf90
merge
chasevoorhees da50036
Merge branch 'hummingbot:main' into development-merged
chasevoorhees e958d25
Revert "Merge branch 'hummingbot:main' into development-merged"
chasevoorhees b0929ab
Update definitions.yml
nkhrs File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,85 @@ | ||
| paths: | ||
| /amm/liquidity/price/: | ||
| post: | ||
| tags: | ||
| - 'cosmos' | ||
| - 'amm/liquidity' | ||
| summary: 'Cosmos: Get the historical pool prices for a pair' | ||
| operationId: 'historical' | ||
| consumes: | ||
| - 'application/json' | ||
| produces: | ||
| - 'application/json' | ||
| parameters: | ||
| - in: 'body' | ||
| name: 'body' | ||
| required: true | ||
| schema: | ||
| $ref: '#/definitions/CosmosPoolPriceRequest' | ||
| responses: | ||
| '200': | ||
| schema: | ||
| $ref: '#/definitions/CosmosPoolPriceResponse' | ||
| /amm/liquidity/add/: | ||
| post: | ||
| tags: | ||
| - 'cosmos' | ||
| - 'amm/liquidity' | ||
| summary: 'Cosmos: Add liquidity to pool' | ||
| operationId: 'add' | ||
| consumes: | ||
| - 'application/json' | ||
| produces: | ||
| - 'application/json' | ||
| parameters: | ||
| - in: 'body' | ||
| name: 'body' | ||
| required: true | ||
| schema: | ||
| $ref: '#/definitions/CosmosAddLiquidityRequest' | ||
| responses: | ||
| '200': | ||
| schema: | ||
| $ref: '#/definitions/CosmosAddLiquidityResponse' | ||
| /amm/liquidity/remove/: | ||
| post: | ||
| tags: | ||
| - 'cosmos' | ||
| - 'amm/liquidity' | ||
| summary: 'Cosmos: Remove liquidity to pool' | ||
| operationId: 'remove' | ||
| consumes: | ||
| - 'application/json' | ||
| produces: | ||
| - 'application/json' | ||
| parameters: | ||
| - in: 'body' | ||
| name: 'body' | ||
| required: true | ||
| schema: | ||
| $ref: '#/definitions/CosmosRemoveLiquidityRequest' | ||
| responses: | ||
| '200': | ||
| schema: | ||
| $ref: '#/definitions/CosmosRemoveLiquidityResponse' | ||
| /amm/liquidity/position: | ||
| post: | ||
| tags: | ||
| - 'cosmos' | ||
| - 'amm/liquidity/' | ||
| summary: 'Cosmos: Get details about position' | ||
| operationId: 'position' | ||
| consumes: | ||
| - 'application/json' | ||
| produces: | ||
| - 'application/json' | ||
| parameters: | ||
| - in: 'body' | ||
| name: 'body' | ||
| required: true | ||
| schema: | ||
| $ref: '#/definitions/CosmosPoolPositionsRequest' | ||
| responses: | ||
| '200': | ||
| schema: | ||
| $ref: '#/definitions/CosmosPoolPositionsResponse' | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| paths: | ||
| /amm/trade/: | ||
| post: | ||
| tags: | ||
| - 'amm' | ||
| - 'cosmos' | ||
| summary: 'Cosmos: Perform a trade on an AMM' | ||
| operationId: 'trade' | ||
| consumes: | ||
| - 'application/json' | ||
| produces: | ||
| - 'application/json' | ||
| parameters: | ||
| - in: 'body' | ||
| name: 'body' | ||
| required: true | ||
| schema: | ||
| $ref: '#/definitions/TradeRequest' | ||
| responses: | ||
| '200': | ||
| schema: | ||
| $ref: '#/definitions/CosmosTradeResponse' |
Oops, something went wrong.
Oops, something went wrong.
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.
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.
please don't override the default docs for these endpoints. currently, the Swagger addition causes duplicate entries in local docs when users go to localhost:8080
if you want to show users how to use cosmos-specific endpoints, I suggest doing so on the doc page for this in the https://github.com/hummingbot/hummingbot-site repo.
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.
hi @fengtality, we have a PR planned for the /hummingbot-site repo with pages for osmosis dex, osmosis chain, update to cosmos chain, and the change to index, as described in the contribution guidelines - just havent done the PR yet, its coming up.