docs: hand-off checklist for the two remaining PI board follow-ups - #5
Merged
Conversation
Adds docs/pi-board-followups.md with precise, environment-specific steps for the two items that need a .NET SDK / running stack (unavailable in the authoring session): 1. Promote board placement to a typed PiObjective.IterationId column — exact model change, `dotnet ef` migration, the PiBoard.cs endpoint rewrite (which also fixes the single-JSON-blob concurrent-write race), optional backfill, and why the frontend/tests need no change. 2. Runtime end-to-end verification — docker compose bring-up, a two-window board/presence/cursor/drag script, the keyboard path, the Teams webhook round-trip, and the runtime-only things to watch (WS 101 through nginx, the access_token hub handshake under auth). Linked from the README documentation section. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01F1g44VLAeYdf3FAoXsXRu7
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.
Documents the two items that need an environment the authoring session didn't have (a .NET SDK for
dotnet ef, and a running stack for end-to-end checks). Everything else from the board/Teams work shipped in #1–#4.New:
docs/pi-board-followups.mdPiObjective.IterationIdcolumn — exact model change, thedotnet ef migrations addcommand, thePiBoard.csendpoint rewrite (which also fixes the single-JSON-blob concurrent-write race), an optional one-time backfill, and why the frontend and existing tests need no change.docker composebring-up (SEED_DEMO_DATA=true), a two-window presence/cursor/drag test script, the keyboard-move path, the Teams webhook round-trip, and the runtime-only things to watch (thewss://…/hubs/board101 through nginx, and theaccess_tokenhub handshake under auth).Linked from the README documentation section.
Docs-only change — no code paths touched.
🤖 Generated with Claude Code
https://claude.ai/code/session_01F1g44VLAeYdf3FAoXsXRu7
Generated by Claude Code