Open
Conversation
…lestone progress visualization
…d milestone progress calculation hook
…for milestone visualization
…rowMilestoneProgressDonut component
…al-dashboard feat(dashboard): implement financial dashboard for receiver escrows
…tics feat: Admin Escrow Analytics Dashboard
Implement Recent Payments view component that displays escrows where the connected wallet is the receiver. Features: - Fetches escrows using useGetEscrowsFromIndexerByRole hook - Displays payment information in responsive card layout - Shows escrow amount, name, milestone description, and status - Handles loading, error, empty, and not connected states - Follows existing UI patterns and design system
Add route for Recent Payments view at /recent-payments. Renders the RecentPaymentsView component to display receiver escrow activity.
- Add loading state during wallet hydration to prevent false disconnected state - Remove USDC fallback assumption for unknown assets, show raw number instead
Add null coalescing operator to prevent runtime errors when escrow.amount is undefined
- Remove unused Card import from service-provider/page.tsx - Remove console.error statements from MilestoneDetailDialog validation function - Remove console.error from both useDetailsEscrow hooks for cleaner production code
- Accept version from main branch - Reinstall dependencies to ensure consistency - Lock file regenerated with npm install --legacy-peer-deps
**Unused imports fixed:** - Remove useEscrowMilestoneProgress from EscrowSummary.tsx - Remove EscrowMilestoneProgressBar from EscrowsTable by-role (unused import) **Semantic HTML improvements:** - Replace orphaned <label> tags with <span> elements in Milestones.tsx for proper semantics **Type safety enhancements:** - Widen formatCurrency currency parameter to accept optional values (currency?: string | null) - Fix dynamic import type in LoadEscrowModal to use LoadEscrowFormProps from correct location **UI/UX improvements:** - Replace empty Progress column with visual placeholder "—" in EscrowsTable by-signer - Prevent rendering of unused EscrowMilestoneProgressBar import in by-role table **Data calculation fixes:** - Fix RelatedEscrows amount calculation for multi-release escrows (sum milestone amounts) - Add helper function getEscrowAmount() to correctly calculate totals - Update formatCurrency calls to handle optional currency values **Documentation updates:** - Document showHeader prop in milestone-progress README - Update API reference to include showHeader in EscrowMilestoneProgressProps interface - Add comment documenting RelatedEscrows current scope limitation **Build:** ✅ Passes without errors
**Progress bar visibility (Major):** - Remove isMultiRelease guard from EscrowSummary.tsx - Show approval progress bar for both single-release and multi-release escrows - Approval mode is supported for all escrow types **Error handling (Major):** - Add isError check in RelatedEscrows component - Handle query failures separately from empty results - Display meaningful error message when data fetch fails - Prevents masking backend/indexer problems as zero-results **Semantic HTML:** - Change label to span in EscrowSummary progress section Build: ✅ Passes all checks
…ayments-view feat: implement Recent Payments view
…e-progress Feature/milestone progress
- Introduced PollarWalletProvider in layout.tsx to support Pollar wallet functionality. - Updated .env.example to include NEXT_PUBLIC_POLLAR_PUBLISHABLE_KEY. - Added @pollar/react dependency in package.json and package-lock.json for wallet integration.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Author
|
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pollar × Trustless Work — Spike Results
What worked
What broke / unknowns
NEXT_PUBLIC_API_KEY(client-side fetch, no server route)config.apiKey, notpublishableKeyRecommendation
Pursue — Pollar successfully abstracts wallet creation, OAuth login, and transaction
signing. UX is significantly simpler for non-crypto users.
Next steps
PollarWalletAdapterinsrc/lib/wallet/pollarAdapter.tsand wire itto the existing escrow flows so both Pollar and the current Stellar wallet are
supported interchangeably.
/api/trustless-work/...) soNEXT_PUBLIC_API_KEYis no longer exposed in the client bundle.PollarWalletProviderfor production use.for onboarding non-crypto users.
Setup
npm install.env.local:npm run dev/pollar-spike