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-mobile): Add delegate to Yoroi banners #3863

Draft
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

michaeljscript
Copy link
Collaborator

@michaeljscript michaeljscript commented Mar 11, 2025

Description / Change(s) / Related issue(s)

Create governance Yoroi delegation banner

Ticket

YV-292

Copy link

codecov bot commented Mar 11, 2025

Codecov Report

Attention: Patch coverage is 46.66667% with 32 lines in your changes missing coverage. Please review.

Project coverage is 22.87%. Comparing base (d70cdbf) to head (d4822df).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #3863      +/-   ##
===========================================
+ Coverage    22.80%   22.87%   +0.06%     
===========================================
  Files         1163     1168       +5     
  Lines        22125    22168      +43     
  Branches      3409     3410       +1     
===========================================
+ Hits          5046     5071      +25     
- Misses       16934    16952      +18     
  Partials       145      145              
Components Coverage Δ
@yoroi/api 100.00% <ø> (ø)
@yoroi/blockchains 100.00% <ø> (ø)
@yoroi/claim 100.00% <ø> (ø)
@yoroi/common 100.00% <100.00%> (ø)
@yoroi/dapp-connector 100.00% <ø> (ø)
@yoroi/exchange 100.00% <ø> (ø)
@yoroi/explorers 100.00% <ø> (ø)
@yoroi/identicon 100.00% <ø> (ø)
@yoroi/links 100.00% <ø> (ø)
@yoroi/notifications 100.00% <ø> (ø)
@yoroi/portfolio 100.00% <ø> (ø)
@yoroi/resolver 100.00% <ø> (ø)
@yoroi/setup-wallet 100.00% <ø> (ø)
@yoroi/staking 68.82% <ø> (ø)
@yoroi/swap 100.00% <ø> (ø)
@yoroi/theme 100.00% <ø> (ø)
@yoroi/transfer 100.00% <ø> (ø)
@yoroi/types ∅ <ø> (∅)
@yoroi/wallet-mobile 9.46% <11.11%> (-0.02%) ⬇️
🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@jorbuedo jorbuedo left a comment

Choose a reason for hiding this comment

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

Dashboard.json probably changed when testing. Will be overridden later when actually used I guess.

@stackchain stackchain added this to the 5.3.1 milestone Mar 12, 2025
@michaeljscript
Copy link
Collaborator Author

Let me fix it

@stackchain stackchain force-pushed the feat/delegation-banner branch from b164c3f to 4f989ba Compare March 12, 2025 08:49
@stackchain
Copy link
Member

stackchain commented Mar 12, 2025

@michaeljscript

  • plug in the right data into the useCases views
  • move yoroi drep constant hash to staking package (export from there)
  • add screens to tx-history / dashboard / governance
  • fix the view is missing the image in the banner
  • add the navigation to the delegate drep - start to US (params) from Banner

@michaeljscript michaeljscript marked this pull request as draft March 12, 2025 13:23
@stackchain stackchain changed the title feat(wallet-mobile): Create governance Yoroi delegation banner feat(wallet-mobile): Add delegate to Yoroi banners Mar 12, 2025
@@ -38,7 +40,7 @@ export const ConsiderDRepToUsTxHistoryBanner = () => {
isStaking: hasStakingKeyRegistered,
dismissedAt,
ptBalance: balance.quantity,
ptMinBalance: 5n * BigInt(ptDecimals),
ptMinBalance: BigInt(minBalanceToDisplayBanner) * BigInt(ptDecimals),
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
ptMinBalance: BigInt(minBalanceToDisplayBanner) * BigInt(ptDecimals),
ptMinBalance: BigInt(minBalanceToDisplayBanner) * BigInt(10n ** ptDecimals),

ptBalance: 0n,
ptMinBalance: 0n,
ptBalance: balance.quantity,
ptMinBalance: 5n * BigInt(ptDecimals),
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
ptMinBalance: 5n * BigInt(ptDecimals),
ptMinBalance: BigInt(minBalanceToDisplayBanner) * BigInt(10n ** ptDecimals),

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants