Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Refactor coinswap #219

Merged
merged 9 commits into from
Aug 10, 2021
Merged

Refactor coinswap #219

merged 9 commits into from
Aug 10, 2021

Conversation

dreamer-zq
Copy link
Contributor

@dreamer-zq dreamer-zq commented Aug 9, 2021 •
edited by chengwenxi
Loading

  • The liquidity pool is named lpt-{index}, and the index is incremented from 1.

@dreamer-zq dreamer-zq self-assigned this Aug 9, 2021
@dreamer-zq dreamer-zq added the modules/coinswap Changes, features and bugs related to the coinswap module. label Aug 9, 2021
chengwenxi
chengwenxi suggested changes Aug 10, 2021
View reviewed changes
modules/coinswap/keeper/pool.go Outdated Show resolved Hide resolved
modules/coinswap/keeper/swap_test.go Outdated Show resolved Hide resolved
modules/coinswap/keeper/swap_test.go Outdated Show resolved Hide resolved
modules/coinswap/migrations/v1110/migrate.go Outdated Show resolved Hide resolved
@chengwenxi chengwenxi added the automerge Automatically merge PR once all prerequisites pass. label Aug 10, 2021
@github-actions github-actions bot merged commit cc9a8f0 into master Aug 10, 2021
@github-actions github-actions bot removed the automerge Automatically merge PR once all prerequisites pass. label Aug 10, 2021
@github-actions github-actions bot deleted the zhiqiang/coinswap-refactor branch August 10, 2021 11:18
@chengwenxi chengwenxi mentioned this pull request Aug 10, 2021
Closed
This was referenced Apr 19, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chengwenxi chengwenxi chengwenxi approved these changes

@zhangyelong zhangyelong zhangyelong approved these changes

Assignees

@dreamer-zq dreamer-zq

Labels
modules/coinswap Changes, features and bugs related to the coinswap module.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dreamer-zq @chengwenxi @zhangyelong