docs: add evidence-gated sprint demo agenda - #1994
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Repository: groupthinking/EventRelay/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Snapshot WarningsEnsure that dependencies are being submitted on PR branches. Re-running this action after a short time may resolve the issue. See the documentation for more information and troubleshooting advice. Scanned FilesNone |
|
@copilot resolve the merge conflicts in this pull request |
There was a problem hiding this comment.
🟡 Changes recommended
Several pull-request statuses are stale, and the task references do not render visibly.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Adds an internal, evidence-gated sprint demo agenda for the UVAI Home-to-Studio workflow.
Changes:
- Defines a time-boxed demo with owners and fallbacks.
- Documents local verification commands and production boundaries.
- Adds closing records and task references.
File summaries
| File | Description |
|---|---|
docs/SPRINT_DEMO_AGENDA.md |
Adds the sprint demonstration agenda and evidence requirements. |
Review details
Suppressed comments (2)
docs/SPRINT_DEMO_AGENDA.md:47
- This required script still calls PR #1984 a draft even though GitHub currently marks it open and non-draft. Make the fallback conditional on live PR state so the presenter does not repeat stale evidence.
**Google sign-in change not yet reviewed:** “The retry behavior is a tested candidate in draft PR #1984. It does not change OAuth credentials, callback policy, or the server authentication gate, and it is not presented as a merged result.”
docs/SPRINT_DEMO_AGENDA.md:58
- These facts are already stale: PR #1981 merged and issue #1975 closed on 2026-09-18, while PR #1984 is open and non-draft. Since the agenda's purpose is accurate evidence reporting, record durable outcomes and require live lookup for statuses that can still change.
1. The landing-page Notion task remains **In progress**. [PR #1976](https://github.com/groupthinking/EventRelay/pull/1976) is closed as a duplicate; [PR #1981](https://github.com/groupthinking/EventRelay/pull/1981) is the canonical landing-page implementation and remains blocked on preview-dependent evidence.
2. Google sign-in recovery is represented by draft [PR #1984](https://github.com/groupthinking/EventRelay/pull/1984). The local test coverage is relevant evidence, but open remote Vercel and E2E failures prevent a readiness claim.
- Files reviewed: 1/1 changed files
- Comments generated: 2
- Review effort level: Balanced
💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| | 4:30–6:00 | **Engineering owner: _assign_** | Explain the Google sign-in recovery behavior: rejected or non-navigating initiation returns the button to a retryable state with generic error copy. | The audience sees the code or test narrative, not a real OAuth completion. | Draft [PR #1984](https://github.com/groupthinking/EventRelay/pull/1984) contains the implementation and its component test. Only demonstrate it as a reviewed outcome after review and its remote check failures are resolved. Until then, explain it as a tested candidate. | | ||
| | 6:00–7:30 | **QA owner: _assign_** | Summarize the local quality gates for the two sprint changes. | The team sees exact commands and the distinction between local tests and remote checks. | Reference [draft QA checklist PR #1993](https://github.com/groupthinking/EventRelay/pull/1993). If that document is still pending review, present the commands directly and do not claim full release readiness. | |
| [1]: https://app.notion.com/p/3dd3c2339c0481439d43eafdeacd4136?pvs=204 "Design sprint landing page task" | ||
| [2]: https://app.notion.com/p/3dd3c2339c0481a2a35dce563f281ab1?pvs=204 "Fix login bug task" | ||
| [3]: https://app.notion.com/p/3dd3c2339c04810ca45be4d289f3e240?pvs=204 "Confirm sprint demo agenda task" | ||
| [4]: https://app.notion.com/p/3dd3c2339c0481019c8fdf22053d0a4d?pvs=204 "Prepare QA checklist task" |
…emo-agenda # Conflicts: # docs/SPRINT_DEMO_AGENDA.md Co-authored-by: groupthinking <154503486+groupthinking@users.noreply.github.com>
Resolved by merging the latest |
Canonical issue
Closes #
Outcome
Adds a time-boxed internal agenda with owners, observable results, evidence commands, and fallback language for every demo segment.
Scope
Risk
Verification
Production evidence
Not applicable. This PR explicitly distinguishes local demonstration evidence from preview and production results.
Agent handoff