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

feat: wallet UI claim rewards #2115

Merged
merged 20 commits into from
Aug 2, 2023
Merged

feat: wallet UI claim rewards #2115

merged 20 commits into from
Aug 2, 2023

Conversation

namgold
Copy link
Contributor

@namgold namgold commented Jul 24, 2023

New Wallet UI

  • new Wallet UI for claiming reward: voting reward and gas refund reward
    • refactor claiming reward hooks to be reuseable
  • integrate gas refund cycle API
  • update texts & images change
  • refactor Icon sprite: add type
  • fix Solana loop call on load page init
  • add Mixpanel event for gas refund
  • fix BE's gasRefundPerCentage -> gasRefundPercentage
  • fix gas refund claim transaction alert & transaction informations
  • fix wallet popup restrict to only able to drag inside of screen
  • reload claim reward info after claimed
  • refactor gas refund use swr => rtk
  • fix remove some duplicated rtk middleware

@kyber-ci-bot
Copy link

Auto Deploy Pull Request

Your domain kyberswap-interface-2115.pr.kyberengineering.io has been created.

Notes

  • Please wait up to 60s, we need to pull your image.
  • This domain only validity until this PR is closed

@namgold namgold force-pushed the feat/wallet-UI-claim-reward branch from 6871211 to 254a328 Compare July 27, 2023 09:30
.env.adpr Outdated Show resolved Hide resolved
@namgold namgold merged commit 6c26663 into main Aug 2, 2023
6 of 7 checks passed
@namgold namgold deleted the feat/wallet-UI-claim-reward branch August 2, 2023 09:46
@kyber-ci-bot
Copy link

Auto Deploy Pull Request

Your domain kyberswap-interface-2115.pr.kyberengineering.io has been revoked because: PR closed.

Notes

  • Please re-open the PR to re-create this domain!

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.

3 participants