Skip to content

feat: Improved sideshift copy UI - #656

Merged
Klakurka merged 1 commit into
masterfrom
feat/improved-sideshift-copy-toast
Jul 24, 2026
Merged

feat: Improved sideshift copy UI#656
Klakurka merged 1 commit into
masterfrom
feat/improved-sideshift-copy-toast

Conversation

@Klakurka

@Klakurka Klakurka commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features

    • Added clearer copy confirmations for payment amounts, addresses, shift IDs, and QR payment details.
    • QR payment areas now display a temporary “Payment copied!” message after copying.
  • Bug Fixes

    • Copy feedback now resets reliably after a short delay, including when leaving the payment view.
  • Tests

    • Added coverage for copying payment details and QR payment links, including confirmation messages and reset behavior.

@Klakurka Klakurka self-assigned this Jul 24, 2026
@Klakurka Klakurka added the enhancement (UI/UX/feature) New feature or request label Jul 24, 2026
@Klakurka Klakurka changed the title feat: Improved copy UI feat: Improved sideshift copy UI Jul 24, 2026
@Klakurka
Klakurka merged commit 0ee5198 into master Jul 24, 2026
2 of 3 checks passed
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: de84f78c-906c-4584-918d-cbb85e1772ba

📥 Commits

Reviewing files that changed from the base of the PR and between bff5d7b and ebc9c37.

📒 Files selected for processing (2)
  • react/lib/components/Widget/AltpaymentWidget.tsx
  • react/lib/tests/components/AltpaymentWidget.test.tsx

📝 Walkthrough

Walkthrough

Changes

Altpayment copy feedback

Layer / File(s) Summary
Copy state, helper, and rendered feedback
react/lib/components/Widget/AltpaymentWidget.tsx
Field and QR copy actions now use state-driven overlays with timed resets and unmount cleanup.
Clipboard feedback validation
react/lib/tests/components/AltpaymentWidget.test.tsx
Tests verify copied values, QR payment URI generation, temporary messages, and feedback restoration.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant AltpaymentWidget
  participant ClipboardAPI
  participant CopyTimeouts
  User->>AltpaymentWidget: Click field or QR copy control
  AltpaymentWidget->>ClipboardAPI: writeText(copy value)
  ClipboardAPI-->>AltpaymentWidget: Copy result
  AltpaymentWidget->>AltpaymentWidget: Set copied feedback state
  AltpaymentWidget->>CopyTimeouts: Schedule feedback reset
  CopyTimeouts-->>AltpaymentWidget: Restore default prompt
Loading

Possibly related PRs

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/improved-sideshift-copy-toast

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

react/lib/components/Widget/AltpaymentWidget.tsx

Parsing error: error TS5012: Cannot read file '/tsconfig.json': ENOENT: no such file or directory, open '/tsconfig.json'.

react/lib/tests/components/AltpaymentWidget.test.tsx

Parsing error: error TS5012: Cannot read file '/tsconfig.json': ENOENT: no such file or directory, open '/tsconfig.json'.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement (UI/UX/feature) New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant