-
Notifications
You must be signed in to change notification settings - Fork 41
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
Allow Closed
pools to be opened
#695
Conversation
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.
Thanks for adding the Initialized
state, so that trading is only allowed after the prediction market pallet allows it. I mentioned some small things.
Co-authored-by: Chralt <chralt98@gmail.com>
Co-authored-by: Chralt <chralt98@gmail.com>
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.
Looks like a charm.
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.
Ok, as you pointed out in Discord: it needs the state of either Initialized
or Active
(so preventing Closed
and Clean
) to join the pool. I couldn't find anything to update. What happens after cleaning up the pool. What happens when I call pool_exit
then?
|
Thanks for the review, guys! 👍 |
This adds the pool status
Initialized
, with which all Cpmm pool start, and adds the functionality to open an initialized pool.The purpose of this is to allow
prediction-markets
to open and close pools depending on market status. The motivation for allowing this is to let a market which is created before it opens to close the pool until the market opens to prevent trading. This will be implemented later. Note that currently, we still callopen_pool
when creating a cpmm pool by default.