Skip to content

feat: improve LP funding confirmation flow#96

Open
newmattock wants to merge 2 commits into
Invoice-Liquidity-Network:mainfrom
newmattock:feat/lp-one-click-funding
Open

feat: improve LP funding confirmation flow#96
newmattock wants to merge 2 commits into
Invoice-Liquidity-Network:mainfrom
newmattock:feat/lp-one-click-funding

Conversation

@newmattock
Copy link
Copy Markdown

Fixes #29

Summary

  • expand the LP funding confirmation modal with amount to send, expected repayment, gross yield in bps and percent, days to due date, and payer reputation score
  • keep approval-required and pre-approved funding paths in the two-step confirmation flow
  • pass fetched payer reputation into the modal and rename the final funding CTA to Fund Now
  • add regression coverage for approval-needed, pre-approved, and funding-detail display paths
  • fix the Vitest @ alias for focused component tests and the current governance Vetoed status build break

Validation

  • npm test -- src/components/tests/FundConfirmModal.test.tsx --reporter verbose
  • npx eslint src/components/FundConfirmModal.tsx src/components/tests/FundConfirmModal.test.tsx vitest.config.ts app/governance/page.tsx
  • npm run build
  • git diff --check
  • local smoke: HEAD /lp returned HTTP 200 on port 3018
  • in-app browser smoke: /lp loaded with LP Dashboard and Fund action present

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build LP funding flow with one-click fund confirmation

1 participant