Skip to content

fix: bonk logo and copy functionality in pnl share#32

Merged
dwjanus merged 1 commit intomainfrom
dwj/pnl-card
Jan 7, 2026
Merged

fix: bonk logo and copy functionality in pnl share#32
dwjanus merged 1 commit intomainfrom
dwj/pnl-card

Conversation

@dwjanus
Copy link
Collaborator

@dwjanus dwjanus commented Jan 6, 2026

Changes

  • fix bonk logo cutoff
  • add base64 conversion for asset images and fallback to local asset url to avoid cors
  • add notification toast on copy error and "Copied" button state for better feedback/usability

Issue

https://linear.app/dydx/issue/ENG-1627/fix-bonk-pnl-card

@dwjanus dwjanus requested a review from a team as a code owner January 6, 2026 17:31
@vercel
Copy link

vercel bot commented Jan 6, 2026

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

Project Deployment Review Updated (UTC)
project-pup Ready Ready Preview, Comment Jan 6, 2026 5:31pm

@dwjanus dwjanus requested a review from jaredvu January 6, 2026 17:32
@dwjanus dwjanus merged commit 81b0358 into main Jan 7, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants