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(staking): add earnings projections and steps to the staking and unstaking modal #14713

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

dev-pvl
Copy link
Contributor

@dev-pvl dev-pvl commented Oct 3, 2024

Description

  • Combined two issues in one PR as they share the same components.
  • Added earnings projection with compound interest link to clarify expected rewards in the staking modal.
  • Updated unstaking steps in the modal.
  • Refactored StakeEthInANutshellModal for cleaner structure and maintainability.

Related Issue

#13939
#13940

Resolve

Screenshots:

Screenshot
Screenshot2
Screenshot3
Screenshot4
Screenshot5
Screenshot6
Screenshot7

content: React.ReactNode;
}

export const InfoCard = ({ heading, content }: InfoCardProps) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use CollapsibleBox for this. See stakeethinnushellmodal (or what's the name:)

@adamhavel
Copy link
Contributor

I think you used an old version of develop for the fork. Please rebase and use the changes we did in the staking modal (using collapsiblebox and list components and so on)

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