-
Notifications
You must be signed in to change notification settings - Fork 0
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
[ADHOC]: draft of page for create boost #23
base: main
Are you sure you want to change the base?
Conversation
</Expandable> | ||
</ParamField> | ||
|
||
<WriteParams /> |
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.
Also uses DeployableOptions
. 🤔
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.
good callout, i'll make a note to remove deployable options, i think i had it tuped that way before i understood cloning/init and was deploying more contracts during create
<ParamField path="allowList" type={<a href="https://boost-protocol.vercel.app/types/AllowList.html">AllowList</a>} /> | ||
<ParamField path="incentives" type={<a href="https://boost-protocol.vercel.app/types/Incentive.html">Incentive[]</a>} required /> | ||
<ParamField path="protocolFee" type="bigint" /> | ||
<ParamField path="maxParticipants" type="bigint" /> |
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.
not sure if best place for it is here, but we need somewhere prominent to point out that budgets will transfer a multiple of maxParticipants worth of assets into the incentive, it can be confusing
core.createBoost()
.Need to add better descriptions and link out to other pages (once they are created)
Looking for feedback mostly before I go further into the examples.