GoodDollar / GoodProtocolUI

UI for the GoodProtocol smart contracts
https://good-swap.vercel.app
GNU General Public License v3.0
7 stars 12 forks source link

Portfolio section: Separate "Claim Rewards" and "Withdraw Liquidity" from staking pools #81

Closed sepu85 closed 2 years ago

sepu85 commented 2 years ago

Bug Description It is currently not possible to Claim Rewards without withdrawing liquidity from staking pool, also UX is not clear about gow many G$s are for the staker to claim his rewards.

Steps to Reproduce

  1. Go to Portfolio
  2. It's assumed user has already staked assets for generating UBI and the rewards program is ongoing.
  3. Click on "Claim Rewards" as there is no option like "Withdraw liquidity"
  4. User can't choose to Claim G$ Rewards without withdrawing his whole liquidit. ... Captura de pantalla 2021-12-16 a las 12 26 00 Captura de pantalla 2021-12-16 a las 12 35 24

Expected Behavior User CAN choose to Claim G$ Rewards without withdrawing his whole liquidity.

Additional Context Add any other context about the problem here (screenshots, whether the bug only occurs only in certain mobile/desktop/browser environments, etc.)

sirpy commented 2 years ago

@sepu85 i think tthis is a mistake and the button should say withdraw next to each protocol at the first section there's a claim button only for rewards

sepu85 commented 2 years ago

i think this is a mistake and the button should say withdraw next to each protocol

I agree, in portfolio section, all rows (staking positions) should have a button saying Withdraw rather than Claim reward

at the first section there's a claim button only for rewards

now I understand, nice. So that's a button that claim ALL rewards from ALL positions?? I believe we should have a dedicated Claim Rewards button per each staking position, but this is a new feature we could add to the backlog (let's talk this before opening a new issue, then).

in issue #42 I stated the following for the G$s reward, now I'm realizing the statement is wrong:

- G$ rewards
How much value your stake has accumulated so far

now, I'm not sure what's supposed to be there, could you @sirpy check configurations and tell me what exactly means that variable? then we ping @adaptivecodeio to update that issue.

I'm guessing those are yearly rewards, but would like you to confirm it before re-opening that issue.

sirpy commented 2 years ago

in the portfolio page at the top it shows your "all accumulated (claimed+unclaimed) rewards from all contracts" in the middle it shows you "claimable rewards" per position it shows the "all accumulated rewards(claimd+unclaimed) from this contract"

Yes we need to reconsider the claim rewards buttons, that was a decision Tomer took.

sirpy commented 2 years ago

@YpetrovAJ Here the user need to have two buttons next to his stake position "Withdraw" and "Claim Rewards" Needs a layout from the designer

YpetrovAJ commented 2 years ago

we've made claim rewards as the check box, it seems for us to look like better on the UX side, can you check it here?