feat(demo): E2E path for web ↔ API ↔ common fixtures (#18) - #26
Closed
kuyacarlo wants to merge 3 commits into
Closed
feat(demo): E2E path for web ↔ API ↔ common fixtures (#18)#26kuyacarlo wants to merge 3 commits into
kuyacarlo wants to merge 3 commits into
Conversation
Populate employees, tasks, and survey admin/dashboard views from common data/utils so the MVP no longer depends on duplicated inline mocks. Co-authored-by: Cursor <cursoragent@cursor.com>
Stack E2E demo (#18) on both sibling PRs so web and API share the @worksight/common contract in one worktree.
Add a runnable demo (pnpm demo /docs/mvp/DEMO.md) so Nest and Next agree on the @worksight/common contract: public /demo page, API CORS on :3001, and an optional NEXT_PUBLIC_USE_API toggle for admin/tasks views. Fixture-backed only. Co-authored-by: Cursor <cursoragent@cursor.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Merged
5 tasks
Contributor
Author
|
Rolled up into #27 ( |
Contributor
Author
|
Rolled into #27 (merged to canary). Closing child PR. |
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
pnpm demo,docs/mvp/DEMO.md,scripts/demo.sh) that boots Nest API + Next web and proves employees/teams/tasks/well-being over the shared@worksight/commoncontract./demopage always fetches Nest;NEXT_PUBLIC_USE_API=truewires/admin/users,/tasks, and/dashboard/tasksto the API (fixture fallback on failure).:3001with CORS; README + handoff updated. Fixture-backed only — not Supabase.Stack
feat/mvp-wire-api-17(MVP: Wire API to @worksight/common types #17 / PR feat(api): wire Nest API to @worksight/common types and fixtures (#17) #24)feat/mvp-wire-web(MVP: Wire web to @worksight/common data/types/utils #16 / PR feat(web): wire dashboard views to @worksight/common (#16) #22) so both tips exist in one worktreefeat/mvp-wire-api-17(stack). After feat(web): wire dashboard views to @worksight/common (#16) #22/feat(api): wire Nest API to @worksight/common types and fixtures (#17) #24 land onfeat/mvp-stabilize, retarget or rebase as needed.Test plan
pnpm --filter @worksight/common build && pnpm --filter @worksight/api build && pnpm --filter @worksight/web buildPORT=3001 node apps/api/dist/main.js→curl/users(15),/teams(5),/tasks(8),/tasks/stats/:id(workLifeBalanceScore)next start→GET /demoHTTP 200 with Nest API / fixture badgespnpm demoend-to-end locally and open http://localhost:3000/demoNEXT_PUBLIC_USE_API=true, confirm/admin/usersshowsdata: Nest APIbadgeCloses #18
Made with Cursor