Pr 26 - #31
Closed
frankosakwe wants to merge 5 commits into
Closed
Conversation
added 5 commits
June 16, 2026 18:21
- Add useRetryQueue hook with complete state management for transaction retries - Implement exponential backoff delays: 1s, 5s, 15s, 30s, 60s - Add localStorage persistence for queue recovery across page reloads - Implement all required functions: enqueue, updateTxHash, markConfirmed, markFailed, retry, purge - Add proper timer cleanup and memory leak prevention - Fix state transitions: pending -> submitted -> confirmed/failed with retrying intermediate state - Add comprehensive test suite with 13 test cases covering serialization and retry logic - Update Playwright configuration with webServer support for E2E testing - Ensure SSR safety with window checks for localStorage access
- Create src/types/retry-queue.ts with QueuedTransaction and UseRetryQueueReturn types - Update useRetryQueue hook to import types from centralized location - Replace all 'any' types in tests with proper QueuedTransaction type - Remove unused 'devices' import from playwright.config.ts - Improve type safety and developer experience with better IntelliSense support
- Replace data: URL with http://localhost:3000 in tests for proper localStorage support - Update reuseExistingServer to use !process.env.CI for better CI/CD compatibility - Fixes SecurityError when accessing localStorage in data: URLs - Ensures fresh server startup in CI environments while reusing in local development
- Replace localhost navigation with about:blank for lighter test setup - Use addInitScript to initialize localStorage before page scripts run - Removes dependency on dev server for localStorage unit tests - Improves test performance and isolation
- Fix lint error by removing any type usage in useRetryQueue tests - Add SSR guards for localStorage access in useWeb3Auth and session service - Replace Math.random() with seeded random generators in FleetGrid and GridMap to fix hydration errors - Update E2E tests to navigate to / instead of about:blank for localStorage access - Add playwright-report and test-results to .gitignore All 20 E2E tests passing, build successful, no lint or type errors
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
2 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.
closes#17