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

Docs edit #2

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

Docs edit #2

wants to merge 12 commits into from

Conversation

qosmonot
Copy link
Contributor

@qosmonot qosmonot commented Nov 2, 2023

Description

This pull request:
rephrases all docs,
fixes typos,
fixes uniform token names,
fixes uniform capitalization for all words

Type of change

Formatting


## Fees

The only fee taken by the vault is **5%** of the yield harvested only in WETH (excluding WAR directly harvested). This fee is used to cover gas fees and will decline as the Vault grows.
The vault charges a **5%** fee exclusively on the WETH portion of the yield (excluding directly harvested $WAR). This fee is used to cover gas expenses and will decrease as the Vault expands.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

is this correct?

@0xmemorygrinder
Copy link
Contributor

Thanks for your interest in our project !
We will rephrase few of your proposal and will merge this PR right after

@qosmonot
Copy link
Contributor Author

qosmonot commented Nov 2, 2023

I added formatting for external references as per the Retype docs (but couldn't check properly that it is correct, sry)

@qosmonot
Copy link
Contributor Author

qosmonot commented Nov 2, 2023

I also suggest editing (done in this last commit) the page titles:
Deposit > Depositing
Withdraw > Withdrawing
Redeem > Redeeming

@qosmonot
Copy link
Contributor Author

qosmonot commented Nov 2, 2023

feel free to discard the last 2 commits if you need

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