-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Coinswap module #4644
Coinswap module #4644
Conversation
…lin/4443-uniswap
…lin/4443-uniswap
For anyone following, development is blocked on #4657 as trading pairs will become a module sub-account |
|
||
// queryParameters returns coinswap module parameter queried for upon success | ||
// or an error if the query fails | ||
func queryParameters(ctx sdk.Context, path []string, req abci.RequestQuery, k Keeper) ([]byte, sdk.Error) { |
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.
return marshalled param struct instead
Fix GetInputAmount formula
* Added query.go that includes the cli commands for querying. * Added transaction subcommands such as add and remove liquidity. * Added tx command for swap order message type. * Added REST API commands for transaction sub commands specific to the coinswap module. * Added REST API commands for the query sub commands. Also added a struct type for query params in querier.go. Incorporated review comment of querying liquidity with a param. Fixed some code problems in tx.go.. * Removed some lines of code that were commented. * Update x/coinswap/client/cli/tx.go Co-Authored-By: colin axner <colinaxner@berkeley.edu> * Update x/coinswap/client/cli/tx.go Co-Authored-By: colin axner <colinaxner@berkeley.edu> * Update x/coinswap/internal/types/querier.go Co-Authored-By: colin axner <colinaxner@berkeley.edu> * Incorporating Colin's and Fede's review comments. * Update x/coinswap/client/rest/query.go Co-Authored-By: colin axner <colinaxner@berkeley.edu> * Update x/coinswap/client/rest/query.go Co-Authored-By: colin axner <colinaxner@berkeley.edu> * Incorporating Fede and Colin's review comments. * Address Colin's comments. * Addressing comments on handling time.parse errors. * Addressed review comments about flags and splitting swap order into buy and sell order. * Addressed Colin's comments. * Addressing comment of removing req.Sender.
As a team, we decided to pause this work as we don't have capacity to maintain this module atm. If someone is interested in resuming work here please get in touch. cc: @sunnya97 |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
this pr has been moved here: cosmos/modules#10 |
closes #4443
To do:
Client, Events, unit tests (keeper, querier, handler), simulation
CLI/Rest being tackled in #4719
I still am in favor of using
sdk.Dec
for the fee parameter, but the swap calculations would need to be rewritten and verified. Using the Uniswap equations until further notice.Targeted PR against correct branch (see CONTRIBUTING.md)
Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
Wrote tests
Updated relevant documentation (
docs/
)Added a relevant changelog entry:
clog add [section] [stanza] [message]
rereviewed
Files changed
in the github PR explorerFor Admin Use: