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.

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

refactor: fix bug in bid refund and remove min_bid_amount #216

Closed
wants to merge 5 commits into from

Conversation

kingcre
Copy link
Contributor

@kingcre kingcre commented Sep 27, 2023

Description

This PR removes min_bid_amount from PublicPosition.

@kingcre kingcre self-assigned this Sep 27, 2023
@github-actions github-actions bot added documentation Improvements or additions to documentation x/liquidamm labels Sep 27, 2023
@blackhatcre
Copy link
Member

In x/liquidamm, min_bid_amount seems to be removed well. But, It has not modified x/exchange and the test (TestOrder_Validate/open_quantity_>_quantity) is failing. Could you please check it?

@kingcre
Copy link
Contributor Author

kingcre commented Oct 10, 2023

@blackhatcre Thank you for figuring it out! The failing test case is fixed in d7da354#diff-c920f3c5561b5b3d87ab5de76fefcb557bf9c9f25b5992bf24dc6364e70e0916R109(it takes a while to load the page).
Let's review this PR again after merging #217.

@kingcre kingcre changed the title refactor: remove min_bid_amount refactor: fix bug in bid refund and remove min_bid_amount Oct 13, 2023
@kingcre kingcre marked this pull request as ready for review October 18, 2023 06:33
@kingcre
Copy link
Contributor Author

kingcre commented Oct 24, 2023

Closing. See #223

@kingcre kingcre closed this Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation x/liquidamm x/liquidstaking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants