-
Notifications
You must be signed in to change notification settings - Fork 0
Feat/phase4 admin 20251029 #37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
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
…ist wiring - AdminApi: metrics, activity, users with safe fallbacks - Hooks: useAdminMetrics, useAdminActivity, useAdminUsers - Admin dashboard: KPI cards, range filters, recent activity table, loading skeletons - Admin users: paginated table with search/filters and skeletons - Tests: admin hooks unit tests; admin E2E smoke (mocked APIs)
…export; add System Health page with polling/backoff and service tiles
… Health completion; mark dashboard/users/settings items done
… lint; display auto-refresh delay via state
… overrides to reflect changes without refetch
…nline Undo banner; implement bulk deactivate/delete
…s for users create/edit flows (mocked)
…health capabilities and usage tips
…mlFor/id to satisfy a11y - Adds id to Notifications select and htmlFor on label - Resolves failing a11y test (findByLabelText)
… Health backoff; add E2E smoke for admin dashboard and users filters feat(health): stub sparkline placeholder in ServiceCard chore(todo): mark completed Admin tasks in TODO.md
… extend webServer timeout test(setup): mock IntersectionObserver and requestIdleCallback in Jest setup to reduce act() warnings
…ng up dev server without E2E env vars - Set reuseExistingServer=false in playwright.ci.config.ts and playwright.config.ts - This prevents accidental redirects to /login and other flakiness when a local dev server is running on :3001 - Verified unit tests still pass and build/lint OK
- Use dedicated port 3101 for E2E server; add script `e2e:start:3101` - Disable webServer reuse; start fresh server with E2E envs - Login: aria-live=assertive for quicker error announcements - Profile E2E: avoid strict-mode conflicts by selecting visible toast - Grading workspace: do not swallow publish errors; let UI show error toast - Add client-side validation for invalid question points in rubric - Relax edge-case test assertion to focus on stay-on-page behavior All focused E2E subsets now pass locally; unit/build/lint remain green.
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.
No description provided.