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

add fraxtal, update description, and include curve lend for Beefy #45

Merged
merged 3 commits into from
May 9, 2024

Conversation

EPETE-EPETE
Copy link
Contributor

not sure if adding fraxtal will cause and issue, I don't see it on any other integrations.

Copy link
Member

@philippe-git philippe-git left a comment

Choose a reason for hiding this comment

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

No it won't cause issues, thanks for double-checking!

Before merging this PR, does Beefy have features that allow to mint/etc crvUSD? Simply allowing the token itself to be used in the app (like any other erc20 token) doesn't qualify for the "crvusd" tag since we want this tag to help crvusd users find out apps specifically providing tools/data/etc for crvusd itself

@EPETE-EPETE
Copy link
Contributor Author

Beefy has vaults building on top of Curve Lend, it appears that having the crvUSD tag is what gets it to show up on the Integrations for Curve Lend.
https://app.beefy.com/vault/curve-lend-crv-crvusd as an example, also have more on Arbitrum.
image

@philippe-git
Copy link
Member

Nice vaults! The fact that "crvusd" tags are shown on the Curve Lend site is an error, we'll have it fixed, thanks for noticing it! Could you use the "lend" tag instead please? We'll make sure this tag is what gets shown the on the Curve Lend site

@EPETE-EPETE
Copy link
Contributor Author

appreciate the help. Should be good now!

@philippe-git philippe-git merged commit 3018a76 into curvefi:main May 9, 2024
@philippe-git
Copy link
Member

philippe-git commented May 9, 2024

Perfect, thanks! The "lend" tag will show up on the Curve Lend Integrations page as expected once the issue I mentioned is fixed (you can follow it here curvefi/curve-frontend#156)

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