Update Frontend#34
Merged
Merged
Conversation
React 19.2.7, react-router 7.18, TanStack Query 5.101, Tailwind 4.3, vite 7.3.6, vitest 4.1, typescript-eslint 8.63, zustand 5.0.14 and friends. Lint, tsc, 170 tests and the production build all pass.
- vite 8.1 + @vitejs/plugin-react 6 (production build now ~0.6s, down from ~10s); vitest 4.1 runs on the same vite. - eslint 10.7 (flat config unchanged; @eslint/js is now an explicit dev dependency since eslint 10 no longer provides it transitively). - typescript 6.0.3 - the ceiling typescript-eslint 8.x supports; TypeScript 7 is out until typescript-eslint catches up. Dropped the deprecated baseUrl from tsconfig (paths are tsconfig-relative). - jsdom 29, lucide-react 1.24, eslint-plugin-react-refresh 0.5. Verified: eslint zero warnings, tsc clean, 170 tests pass, build OK.
11 browser tests across 4 journeys, driven against the Vite dev server
with every /api/v1 endpoint mocked in-page (no backend required):
- dashboard: stats render from analytics data, root redirect,
scheduler stop hits POST /scheduler/stop
- settings: form loads server values (save disabled), editing
PHPSESSID enables save and the PUT carries the new value,
test-session flow reports the user
- giveaways: list renders, enter posts to /giveaways/{code}/enter,
search narrows the list
- navigation: sidebar reaches all 7 pages, unknown routes fall back
e2e/mocks.ts centralizes the fixtures and returns a request log for
call assertions. New scripts: test:e2e, test:e2e:ui. Playwright
artifacts gitignored.
Known follow-up (found by these tests): History.tsx and Analytics.tsx
throw an unhandled TypeError and blank the page when the API returns
a partial/unexpected shape - they need defensive guards.
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
1 participant
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.
Update version
Add E2E tests front only