Skip to content

Updated key features to include profit sharing explanations#770

Open
paradigmxnic wants to merge 5 commits into
mainfrom
paradigmxnic-patch-2
Open

Updated key features to include profit sharing explanations#770
paradigmxnic wants to merge 5 commits into
mainfrom
paradigmxnic-patch-2

Conversation

@paradigmxnic
Copy link
Copy Markdown
Collaborator

No description provided.

@paradigmxnic paradigmxnic requested review from a team as code owners September 17, 2025 08:05
@github-actions
Copy link
Copy Markdown

Copy link
Copy Markdown
Collaborator Author

@paradigmxnic paradigmxnic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$ sign did a weird thing in html preview

@github-actions
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

Comment thread fern/pages/vaults/key-features.mdx Outdated
Comment on lines +42 to +44
- Profit = $100 ⇒ profit share = $10 ⇒ vault owner receives 10 vault tokens.
- The user’s withdrawal burns 90 tokens (redeemed for $180) and transfers 10 tokens to the vault owner.
The vault owner can later redeem or keep those tokens.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

vault owner should receive 5 tokens, not 10, because the token price increased to 2

the user should burn 95 tokens, not 90

redeemed for $190, not 180

and should transfer 5 tokens to the vault owner, not 10

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise the owner will receive the equivalent of $20 as profit share

Comment thread fern/pages/vaults/key-features.mdx Outdated
Co-authored-by: Mattia D'Antonio <mdantonio@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants