Skip to content

Aded claim Reward on user profile#107

Merged
AndresRamirez9912 merged 4 commits into
mainfrom
feat/add-claim-rewards
Jul 16, 2025
Merged

Aded claim Reward on user profile#107
AndresRamirez9912 merged 4 commits into
mainfrom
feat/add-claim-rewards

Conversation

@AndresRamirez9912
Copy link
Copy Markdown
Collaborator

Description

These changes add the following:

  • Show user rewards on the main page
  • Show user rewards on the profile page
  • Show a table with the rewards per validator and its button, Claim
  • Show a button to claim all rewards

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Documentation (updates documentation on the project)
  • chore (Updates on dependencies, gitignore, etc)
  • test (For updates on tests)

How Has This Been Tested?

This code has been tested locally

Evidences

Initial rewards

image

Claiming

(We have spinning meaning loading)
image

Claimed

(The reward balance has changed)
image

@vercel
Copy link
Copy Markdown

vercel Bot commented Jul 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kii-explorer-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 16, 2025 4:04pm

Comment thread src/services/contracts/distribution.ts Outdated
const address = signer.address;
const distPrecompile = getDistributionPrecompileEthersV6Contract(signer);

const tx = await distPrecompile.claimRewards(address, 50);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Its safe to use an arbitrary value here?

@AndresRamirez9912 AndresRamirez9912 merged commit d991994 into main Jul 16, 2025
5 checks passed
@AndresRamirez9912 AndresRamirez9912 deleted the feat/add-claim-rewards branch July 16, 2025 17:11
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