feat: E2E tests and UX improvements (#900-#903)#969
Open
lolandriley-wq wants to merge 4 commits into
Open
Conversation
- Install Playwright testing framework - Add playwright.config.ts for Chrome and Firefox - Create send-money-flow.spec.ts with: * Happy path test: wallet connect → amount → corridor → confirm * Error test: insufficient balance detection * Error test: rate limit detection - Update .gitignore for Playwright artifacts - Add e2e npm scripts for running tests
…Tel tracing - Add Report Error button with clipboard copy functionality - Capture error details in OTel traces when available - Add handleReportError method for error reporting - Add tertiary button styling for report error button - Improve error details formatting for user support - Support graceful fallback to prompt() if clipboard API unavailable
- Create CorridorComparison.tsx with corridor data fetching - Display corridors sorted by effective fee percentage - Add filters for source and destination countries - Show FX rates and estimated delivery times - Responsive table design with hover states - Emit onSelect callback when user chooses a corridor - Include loading and error states - Add comprehensive styling with CSS variables support
- Create SkeletonLoader.tsx with reusable skeleton components * SkeletonLine for text content * SkeletonBlock for rectangular content * SkeletonTable for tabular data * SkeletonList for lists - Add shimmer animation with CSS keyframes - Apply skeletons to: * RemittanceList loading state * TransactionHistory with conditional skeletons * CorridorAnalytics with block/line skeletons - Responsive skeleton grid for mobile devices - Smooth transition from skeleton to content
|
@lolandriley-wq is attempting to deploy a commit to the Harold's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@lolandriley-wq 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! 🚀 |
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
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.
Summary
This PR addresses 4 frontend issues with end-to-end testing and user experience improvements:
Commits
Add end-to-end Playwright tests for core send money flow #902 - E2E Tests for Send Money Flow
Add React Error Boundary recovery UI #903 - Error Boundary Recovery UI
Add corridor fee comparison table #900 - Corridor Fee Comparison Table
Add loading skeletons for async data #901 - Loading Skeleton States
Acceptance Criteria Met
Testing
Run locally:
Closes #901
Closes #902
Closes #903
Closes #900