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

Copy does not make sense #60

Open
iurage opened this issue Mar 8, 2023 · 4 comments
Open

Copy does not make sense #60

iurage opened this issue Mar 8, 2023 · 4 comments
Labels
enhancement New feature or request

Comments

@iurage
Copy link

iurage commented Mar 8, 2023

"Total number of underlying tokens need to distribute to all recipients:" - doesnt make senes to me here.

4 PSY is the total amount of PSY that is being used as collateral for underwriting the options right? How about changing this to:

"Total number of underlying tokens required to mint options"

Screenshot 2023-03-08 at 2 36 58 PM

@iurage iurage added the enhancement New feature or request label Mar 8, 2023
@iurage
Copy link
Author

iurage commented Mar 8, 2023

Part 2:

"4 PsyOptions option contracts" - Is PsyOptions here referring to the underlying token name? or is it the type of contract - getting confused here

Screenshot 2023-03-08 at 2 39 27 PM

@iurage
Copy link
Author

iurage commented Mar 8, 2023

Spelling error for sufficient here
Screenshot 2023-03-08 at 3 17 05 PM

@parhim
Copy link
Contributor

parhim commented May 10, 2023

@iurage with regards to Total number of underlying tokens need to distribute to all recipients

it is possible to distribute first come first served options if you chose to use less underlying,
we need to change the copy to account for that;

@iurage
Copy link
Author

iurage commented May 15, 2023

Sounds good, whichever is clearer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants