Skip to content
This repository was archived by the owner on May 1, 2023. It is now read-only.

fix a wrong url in code-repositories.md - #518

Merged
lehnberg merged 2 commits into
yearn:masterfrom
gzliudan:master
Aug 5, 2021
Merged

fix a wrong url in code-repositories.md#518
lehnberg merged 2 commits into
yearn:masterfrom
gzliudan:master

Conversation

@gzliudan

Copy link
Copy Markdown
Contributor

No description provided.

@lehnberglehnberg left a comment

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.

Thanks for the PR @gzliudan! Please see suggested changes above.

Comment threaddevelopers/code-repositories.md Outdated
- [Vaults](https://github.com/yearn/yearn-protocol/tree/develop/contracts/vaults)
- [Strategies](https://github.com/yearn/brownie-strategy-mix)
- [iToken Wrappers](https://github.com/yearn/brownie-strategy-mix)
- [iToken Wrappers](https://github.com/yearn/itoken)

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.

Actually, would be even better if we linked to:

[Token Wrappers](https://github.com/yearn/brownie-wrapper-mix)

here instead, as far as I know the iToken repo is not active any longer.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

I saw Strategies is linked to same link, so I think the link of iToken Wrappers is wrong. Sorry to trouble you.

@gzliudangzliudan closed this Aug 5, 2021
@lehnberg

Copy link
Copy Markdown
Contributor

nothing to be sorry for @gzliudan, feel free to push a fixed commit and I'll merge it!

@lehnberglehnberg reopened this Aug 5, 2021
@gzliudan

Copy link
Copy Markdown
ContributorAuthor

Ok, the link is changed to Token Wrappers from iToken Wrappers according to your suggest. I'm new to yearn. Thanks for your help.

@lehnberglehnberg left a comment

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.

Looks great, thanks for this, and welcome to Yearn! 👍

@lehnberg
lehnberg merged commit 2daf273 into yearn:masterAug 5, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@gzliudan@lehnberg