Repository navigation
fix: scope dashboard commitments by owner and show per-asset totals - #2115
Conversation
|
@abdulraheemabdullahi30-tech is attempting to deploy a commit to the 1nonly's projects Team on Vercel. A member of the Team first needs to authorize it. |
c3a18ce to
44e100f
Compare
Strips the payload this branch picked up (deleted api.js, truncated next.config.js, unhooked package.json). No functional change to the branch's own work.
|
@abdulraheemabdullahi30-tech Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
Hi @abdulraheemabdullahi30-tech, thanks for the PR. We can't merge it yet: it adds these two lines to These entries are a known indicator of a malware campaign that infects developer machines and auto-pushes hidden code to their repositories. The malware adds them to hide its own helper scripts. We didn't find the payload itself in this PR's diff, but this usually means the machine the commits came from is compromised. What to do before we can merge this PR
Thanks for your contribution, and sorry for the hassle. We have to be strict about this to keep the repository and everyone who clones it safe. |
Overview
This PR fixes the dashboard so that "Total locked" and "Projected yield" no longer add USDC, XLM and EURC amounts into one meaningless number, and so the commitment list is scoped to the connected wallet. It also makes the "Ready to settle" count always visible and formats remaining time in hours/minutes during the final day.
Related Issue
Changes
📊 Per-asset totals and owner scoping
[MODIFY]
src/app/dashboard/dashboard.test.tsxuseWalletacross two addresses and the disconnected case.[MODIFY]
src/app/globals.cssVerification Results
Closes #2082