Open
Conversation
- Consolidate all backend API endpoints under /api/v1/ prefix - Move summary, results, trigger into api_v1 router - Change trigger from GET to POST (has side effects) - Remove standalone /monitors routes and root handler - Add tower-http ServeDir for serving frontend static files with SPA fallback - Scaffold React 19 + TypeScript + Vite frontend with Tailwind CSS v4 - Implement full-featured dashboard: - Monitor list with status badges, tag filtering, auto-refresh - Monitor detail page with config summary, results timeline, trigger/edit/delete - Create/edit forms supporting single-step and multi-step monitors - Expectations UI with field/operation/value rows and contextual hints - Headers, alerts, tags management with add/remove - Client-side validation and server error display - Optimistic locking for edits (version-based conflict detection) - Add 45 frontend tests (Vitest + React Testing Library): - API client tests with mocked fetch - Monitor list page tests (rendering, filtering, error states) - Monitor form tests (validation, submission, mode toggling) - Monitor detail page tests (config display, trigger, delete) - Update Dockerfile with Node.js frontend build stage - Update .gitignore for frontend artifacts https://claude.ai/code/session_01U5LQPYdkkUjS7h9UH2bz1d
- Fix rustfmt formatting in api_v1/mod.rs and web_server/mod.rs - Add frontend_test job to test.yaml workflow (npm ci, build, test) https://claude.ai/code/session_01U5LQPYdkkUjS7h9UH2bz1d
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.
No description provided.