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

[Bug]: "Accumulated Rewards" value is growing after the risk bundle was burned #203

Open
1 task done
rapidddenis opened this issue Apr 1, 2023 · 5 comments
Open
1 task done
Labels
bug bounty bug Something isn't working

Comments

@rapidddenis
Copy link

What happened?

As a DIP staker I expect my rewards are accumulating only while risk bundle is active

What operating system are you seeing the problem on?

Mac OS

What browsers are you seeing the problem on?

Safari

What wallet extension are you using?

Metamask

Rules

  • I agree to follow this project's rules
@rapidddenis rapidddenis added bug Something isn't working bug bounty labels Apr 1, 2023
@rapidddenis rapidddenis changed the title [Bug]: "Accumulated Rewards" value is growing after the risk bundle was closed and all USDT was unstaked [Bug]: "Accumulated Rewards" value is growing after the risk bundle was burned Apr 1, 2023
@doerfli
Copy link
Contributor

doerfli commented Apr 3, 2023

this could happen if the bundle was burned before expiration. but the value can never be collected (as it is calculated only for display and the state is not taken into account).

https://github.com/etherisc/gif-staking-ui/blob/develop/src/backend/staking_contract.ts#L210

@doerfli doerfli added this to the v1.1 milestone Apr 3, 2023
@rapidddenis
Copy link
Author

It is possible to accumulate and collect after the bundle was burned: etherisc/depeg-contracts#31

@doerfli
Copy link
Contributor

doerfli commented Apr 3, 2023

needs to be in sync with solution for etherisc/depeg-contracts#31

@doerfli doerfli removed this from the v1.1 milestone Apr 4, 2023
@susanneetherisc
Copy link

Is this a duplicate of #31? @doerfli will look into when back from leave

@doerfli
Copy link
Contributor

doerfli commented May 11, 2023

No, this is not a duplicate, but it depends on the outcome of etherisc/depeg-contracts#31 to determine if the current behavior of the webapp needs to change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bounty bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants