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

Error in the display of yields: APY instead of APR #840

Closed
Kolibri1990 opened this issue May 11, 2021 · 10 comments
Closed

Error in the display of yields: APY instead of APR #840

Kolibri1990 opened this issue May 11, 2021 · 10 comments
Labels
kind/feature New feature request triage/accepted Triage has been accepted

Comments

@Kolibri1990
Copy link

What happened:

In the new wallet the display of the returns is still wrong. The returns are APR and not APY.

What you expected to happen:

You only have to change APY in APR... thats all.

How to reproduce it (as minimally and precisely as possible):

Open the wallet.

What are your environment parameters?

  • DeFi Wallet Version: All versions
  • Operating System (OS): All OS

Anything else we need to know?:

@Kolibri1990 Kolibri1990 added the kind/bug Something isn't working label May 11, 2021
@defichain-bot defichain-bot added the needs/triage Needs triage label label May 11, 2021
@defichain-bot
Copy link

@Kolibri1990: Thanks for opening an issue, it is currently awaiting triage.

The triage/accepted label can be added by foundation members by writing /triage accepted in a comment.

In the meantime, you can:

  1. Checkout DeFiChain’s Github issue page to see if your issue has already been reported
  2. Submit any logs if you have them, this will greatly expedite the process for us.
  3. You can also join our Telegram or Reddit community channels.
Details

I am a bot created to help the DeFiCh developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the DeFiCh/oss-governance-bot repository.

@izzycsy
Copy link
Contributor

izzycsy commented May 12, 2021

Hi @Kolibri1990 , thanks for your feedback, the team is aware and are currently investigating.

@izzycsy izzycsy added triage/accepted Triage has been accepted kind/feature New feature request and removed needs/triage Needs triage label triage/accepted Triage has been accepted labels May 12, 2021
@defichain-bot defichain-bot added needs/triage Needs triage label and removed kind/bug Something isn't working labels May 17, 2021
@diegorodriguezv
Copy link

I noticed the web site (https://defichain.com/dex/) shows the rate as APR but the App and the screenshots show APY.
I don't know which is correct. Could someone explain why one or the other?
I have looked around but I can't find which is the number of periods for the calculation. Is it monthly? Daily? Each 12 hours? (https://app.cakedefi.com/ says Rewards every 12 hours)

@Kolibri1990
Copy link
Author

Hey @diegorodriguezv

As with cake, the return on LM on the DeFi Wallet is an APR and not an APY.

It is wrong on the DEX page (https://dex.defichain.com/mainnet/pool) and also the DeFi Wallet.

The values are right but the naming is wrong.

@izzycsy
Copy link
Contributor

izzycsy commented May 20, 2021

Hi @diegorodriguezv you can ask the community DeFiChain Telegram (EN) on such enquiries.

Thanks @Kolibri1990 !

@izzycsy
Copy link
Contributor

izzycsy commented May 28, 2021

Hi @Kolibri1990 @diegorodriguezv @wmbst @adrian-schnell v2.4.0 is out, this is a mandatory update. Please read through the release notes before updating. This comment is to notify you about the update.

@sandrich
Copy link

When is this going to happen? APY is wrong and misleading.

@nick-preda
Copy link

so to be clear, app should be also showing APR, as the website already shows correctly that is APR

@izzycsy
Copy link
Contributor

izzycsy commented Jun 1, 2021

Hi @sandrich , the team is working on it, kindly refer to DeFiChain Announcements (EN) for the latest updates. Thank you.

@thedoublejay
Copy link
Member

Hi, we have updated this on the latest app release v2.4.2, please have a look. thank you - https://defichain.com/downloads/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature request triage/accepted Triage has been accepted
Projects
None yet
Development

No branches or pull requests

7 participants