Skip to content

fix: live address validation, cursor labels, SVG export, cursor pagination - #716

Open
ink-mami wants to merge 1 commit into
Stellar-split:mainfrom
ink-mami:feature/681-680-671-670-hook-and-component-fixes
Open

fix: live address validation, cursor labels, SVG export, cursor pagination#716
ink-mami wants to merge 1 commit into
Stellar-split:mainfrom
ink-mami:feature/681-680-671-670-hook-and-component-fixes

Conversation

@ink-mami

Copy link
Copy Markdown

Summary

  • useAddressValidation: adds a live mode (debounced 300ms, auto-validates via onAddressChange) and a detailed errorReason enum (too_short | invalid_checksum | wrong_prefix | null) instead of a plain boolean.
  • CursorOverlay: renders each collaborator's display name beside their cursor badge, fading out after 3s of inactivity and reappearing immediately on movement.
  • FlowDiagram: adds a "Download SVG" button that serializes the rendered diagram to flow-diagram.svg.
  • useInfiniteInvoices (useInfiniteInvoices: Migrate to cursor-based pagination #681): verified already fully cursor-based (uses cursor/nextCursor end-to-end via /api/invoices), no code change needed.

Closes: #680
Closes: #671
Closes: #670
Closes: #681

Validation performed

  • Manual review of all changed logic against each issue's acceptance criteria.
  • npm ci currently fails in this environment due to a pre-existing broken dependency (@typescript-eslint/plugin@^6.21.0 not found on the registry), unrelated to this change, so lint/typecheck could not be executed locally — flagging for maintainers.

…xport

- useAddressValidation: add live mode (debounced 300ms) and errorReason
  enum (too_short | invalid_checksum | wrong_prefix | null)
- CursorOverlay: show collaborator display name, fading out after 3s
  of cursor inactivity
- FlowDiagram: add "Download SVG" button to export the diagram

Closes: Stellar-split#680
Closes: Stellar-split#671
Closes: Stellar-split#670
Closes: Stellar-split#681
@drips-wave

drips-wave Bot commented Aug 31, 2026

Copy link
Copy Markdown

@ink-mami 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! 🚀

Learn more about application limits

@vercel

vercel Bot commented Aug 31, 2026

Copy link
Copy Markdown

@ink-mami is attempting to deploy a commit to the kingsman-99's projects Team on Vercel.

A member of the Team first needs to authorize it.

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

Labels

None yet

Projects

None yet

1 participant