Skip to content

[Human App FE] Staking Requirements - #3725

Merged
dnechay merged 17 commits into
feat/staking-requirementfrom
kb/3679-staking-info
Dec 22, 2025
Merged

[Human App FE] Staking Requirements#3725
dnechay merged 17 commits into
feat/staking-requirementfrom
kb/3679-staking-info

Conversation

@KirillKirill

Copy link
Copy Markdown
Contributor

Issue tracking

Closes #3679

Context behind the change

Adds staking requirement block on the profile page

How has this been tested?

locally

Release plan

regular

Potential risks; What to monitor; Rollback plan

not found

@KirillKirill
KirillKirill requested a review from dnechay December 18, 2025 14:24
@KirillKirill KirillKirill self-assigned this Dec 18, 2025
@vercel

vercel Bot commented Dec 18, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

5 Skipped Deployments
Project Deployment Review Updated (UTC)
faucet-frontend Ignored Ignored Preview Dec 22, 2025 3:04pm
faucet-server Ignored Ignored Preview Dec 22, 2025 3:04pm
human-app Skipped Skipped Dec 22, 2025 3:04pm
human-dashboard-frontend Skipped Skipped Dec 22, 2025 3:04pm
staking-dashboard Skipped Skipped Dec 22, 2025 3:04pm

@KirillKirill KirillKirill changed the title Kb/3679 staking info [Human App FE] Staking Requirements Dec 18, 2025
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 18, 2025 14:29 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 18, 2025 14:29 Inactive
@dnechay
dnechay requested a review from flopez7 December 18, 2025 15:05
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 18, 2025 15:09 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 18, 2025 15:09 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 18, 2025 15:21 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 18, 2025 15:21 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 18, 2025 15:31 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 18, 2025 15:31 Inactive

@dnechay dnechay left a comment

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.

Codewise lgtm
However, I would wait for @flopez7 review and tests since I don't have full local setup atm to verify changes

@flopez7 flopez7 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overall looks good. Just two small details. When you add your API keys, the profile status page updates, but the “Tasks” section remains disabled. I have to refresh the page manually to enable it. The same thing happens in reverse. When you remove your API keys and you don't have any staking, the “Tasks” section remains enabled and I can access it.

Also, when checking the staking summary with the refresh button, if staking eligibility is disabled and the user doesn't have stake, we have isStaked=false but user.is_stake_eligible=true and is sending summary and refresh calls. Can we fix it? I'm thinking on skip refresh call when user.is_stake_eligible=true

@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 19, 2025 12:14 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 19, 2025 12:14 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 19, 2025 12:31 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 19, 2025 12:31 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 19, 2025 13:45 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 19, 2025 13:45 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 22, 2025 13:33 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 22, 2025 13:33 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 22, 2025 13:47 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 22, 2025 13:47 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-app December 22, 2025 15:04 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard December 22, 2025 15:04 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend December 22, 2025 15:04 Inactive
@dnechay
dnechay merged commit 93e96b2 into feat/staking-requirement Dec 22, 2025
9 checks passed
@dnechay
dnechay deleted the kb/3679-staking-info branch December 22, 2025 15:08
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