Author SHA1 Message Date
Tirsvad f161859225 Resolve pending commit links (MIL-002)
English comments / english-comments (pull_request) Successful in 26s
2026-10-24 23:57:31 +08:00
Tirsvad 5fe4ef0367 Add the Favorite Movie page, styles and manual checklist (MIL-002)
Page uses S01's supplied HTML wrapped in an HTML5 document. The checklist was
run in the built-in browser pane; RC-006 records the review.

Task: MIL-002#1
Task: MIL-002#2
Task: MIL-002#3
Task: MIL-002#4
2026-10-24 23:57:29 +08:00
Tirsvad 83b07742a8 Merge pull request 'MIL-001 Foundation' (#4) from mil-001-foundation into main
Publish accepted milestones / publish (push) Successful in 21s
TirSystem/github-action: Sync GitHub mirror metadata / sync-metadata (push) Successful in 8s
2026-10-24 17:55:52 +02:00
9 changed files with 177 additions and 51 deletions
+3
View File
@@ -17,3 +17,6 @@ tmp/
# AI prompt log (public repository)
/docs/ai-prompts/
# Local preview configuration (a throwaway static server for screenshots)
/.claude/launch.json
+1 -1
View File
@@ -15,7 +15,7 @@ document of a type. `Primary File` may contain a glob (e.g.
| SA | Stakeholder Analysis | docs/stakeholder-analysis.md | 002 |
| PP | Project Plan | docs/project-plan.md | 002 |
| MIL | Milestone / Gateway | docs/milestones/mil-*.md | 004 |
| RC | Review Record | docs/sqa/reviews/rc-*.md | 006 |
| RC | Review Record | docs/sqa/reviews/rc-*.md | 007 |
| TM | Traceability Matrix | docs/sqa/traceability-matrix.md | 002 |
## Languages
+48
View File
@@ -0,0 +1,48 @@
# SQA Review Record
## Metadata
| Key | Value |
| --- | --- |
| ID | RC-006 |
| CrossReference | [MIL-002] |
## Version History
| Date | Status | Author | Reviewer | Change | Commit |
| --- | --- | --- | --- | --- | --- |
| 2026-10-24 | Accepted | Jens Tirsvad Nielsen | AI as stand-in for S01 | Initial version | [5fe4ef0] |
---
## Artifact Under Review
- Instance reviewed: [MIL-002] deliverable: `src/index.html`, `src/styles.css`, `tests/manual-checklist.md`
- Checklist used: the Go / No-Go criteria of [MIL-002]. No QC checklist exists for HTML source in the framework (`framework/qc/code/` has none), so the milestone criteria stand in as the checklist.
- Scope: full review
- Language and domain: n/a (a deliverable, not a document of a PO-language type)
- Language reviewer: none
- Mode: autonomous, not independent (the reviewer is `AI as stand-in for S01`)
## Checklist Results
| # | Criterion | Status | Evidence/Notes |
| --- | --- | --- | --- |
| 1 | One h1 "The Best Movies According to Tirsvad", one h2, one hr | Pass | Read from the DOM in the browser pane: 1 h1, 1 h2 "My top 3 movies of all-time.", 1 hr. |
| 2 | Three h3 in order 13 element, 5th element, Drive, each directly followed by a p | Pass | DOM read: the next sibling of each h3 is a P. The wording is S01's supplied HTML, unchanged. |
| 3 | HTML5 with `lang`, `charset` and `viewport`, no external request | Pass | `<!DOCTYPE html>`, `lang="en"`, charset and viewport meta present; the resource list holds only `styles.css`. No W3C validator was run (no network tool for it). |
| 4 | Every item of `tests/manual-checklist.md` marked pass, checked in the browser pane | Pass | Eight items run on 2026-10-24, including a 375 px wide viewport without sideways scroll. |
| 5 | Reviewed against the assignment and the checklist | Pass | The assignment asks for h1, h2, hr and at least three h3 each with a p; all are present. |
## Overall Verdict
Go — all five criteria pass. This is a stand-in review, not an independent one, and S01 can challenge it in the final summary.
## Action Items
| Action | Owner | Due |
| --- | --- | --- |
| None | S01 | n/a |
---
[MIL-002]: ../../milestones/mil-002-movie-page.md
[5fe4ef0]: https://git.tirsystem.com/Tirsvad-Udemy-100-days-of-code/041-web-movie-ranking/commit/5fe4ef03673e70026a0667a2585f2d9decb6a2c2
+53 -50
View File
@@ -1,50 +1,53 @@
# Traceability Matrix
## Metadata
| Key | Value |
| --- | --- |
| ID | TM-001 |
| CrossReference | [BC-001] |
## Version History
| Date | Status | Author | Reviewer | Change | Commit |
| --- | --- | --- | --- | --- | --- |
| 2026-10-24 | Accepted | Jens Tirsvad Nielsen | AI as stand-in for S01 | Initial version | [e8d24b2] |
---
## Purpose
Tracks backward/forward links between artifact instances so that the Business Case's
cross-artifact traceability success criterion is measurable. A row is added or
updated whenever an artifact instance is created or reviewed.
## Traceability Table
| Artifact Instance | Type | Language | Domain | Upstream (Backward Link) | Downstream (Forward Link) | Last Reviewed (RC-ID) |
| --- | --- | --- | --- | --- | --- | --- |
| [BC-001] | BC | en | it | - | [SA-001], [PP-001] | [RC-001] |
| [SA-001] | SA | en | it | [BC-001] | [PP-001] | [RC-002] |
| [PP-001] | PP | en | it | [BC-001], [SA-001] | [MIL-001], [MIL-002], [MIL-003] | - |
| [MIL-001] | MIL | en | it | [BC-001], [PP-001] | [MIL-002] | [RC-003] |
| [MIL-002] | MIL | en | it | [BC-001], [PP-001] | [MIL-003] | [RC-004] |
| [MIL-003] | MIL | en | it | [BC-001], [PP-001] | - | [RC-005] |
## Coverage Notes
No use case, domain model or design artifact exists: the project is too small for them. `-` in Last Reviewed for PP-001 means the Project Plan has no checklist; S01 accepts it (the AI as stand-in). `-` in Upstream means foundational; `-` in Downstream means nothing is built on it yet.
---
[BC-001]: ../business-case.md
[SA-001]: ../stakeholder-analysis.md
[PP-001]: ../project-plan.md
[MIL-001]: ../milestones/mil-001-foundation.md
[MIL-002]: ../milestones/mil-002-movie-page.md
[MIL-003]: ../milestones/mil-003-documentation.md
[RC-001]: ./reviews/rc-001-business-case.md
[RC-002]: ./reviews/rc-002-stakeholder-analysis.md
[RC-003]: ./reviews/rc-003-mil-001-foundation.md
[RC-004]: ./reviews/rc-004-mil-002-movie-page.md
[RC-005]: ./reviews/rc-005-mil-003-documentation.md
[e8d24b2]: https://git.tirsystem.com/Tirsvad-Udemy-100-days-of-code/041-web-movie-ranking/commit/e8d24b2d32ce1ef8276123b0efa05b8a15e27d52
# Traceability Matrix
## Metadata
| Key | Value |
| --- | --- |
| ID | TM-001 |
| CrossReference | [BC-001] |
## Version History
| Date | Status | Author | Reviewer | Change | Commit |
| --- | --- | --- | --- | --- | --- |
| 2026-10-24 | Deprecated | Jens Tirsvad Nielsen | AI as stand-in for S01 | Initial version | [e8d24b2] |
| 2026-10-24 | Accepted | Jens Tirsvad Nielsen | AI as stand-in for S01 | Added RC-006 for the MIL-002 deliverable | [5fe4ef0] |
---
## Purpose
Tracks backward/forward links between artifact instances so that the Business Case's
cross-artifact traceability success criterion is measurable. A row is added or
updated whenever an artifact instance is created or reviewed.
## Traceability Table
| Artifact Instance | Type | Language | Domain | Upstream (Backward Link) | Downstream (Forward Link) | Last Reviewed (RC-ID) |
| --- | --- | --- | --- | --- | --- | --- |
| [BC-001] | BC | en | it | - | [SA-001], [PP-001] | [RC-001] |
| [SA-001] | SA | en | it | [BC-001] | [PP-001] | [RC-002] |
| [PP-001] | PP | en | it | [BC-001], [SA-001] | [MIL-001], [MIL-002], [MIL-003] | - |
| [MIL-001] | MIL | en | it | [BC-001], [PP-001] | [MIL-002] | [RC-003] |
| [MIL-002] | MIL | en | it | [BC-001], [PP-001] | [MIL-003] | [RC-004], [RC-006] |
| [MIL-003] | MIL | en | it | [BC-001], [PP-001] | - | [RC-005] |
## Coverage Notes
No use case, domain model or design artifact exists: the project is too small for them. `-` in Last Reviewed for PP-001 means the Project Plan has no checklist; S01 accepts it (the AI as stand-in). `-` in Upstream means foundational; `-` in Downstream means nothing is built on it yet.
---
[BC-001]: ../business-case.md
[SA-001]: ../stakeholder-analysis.md
[PP-001]: ../project-plan.md
[MIL-001]: ../milestones/mil-001-foundation.md
[MIL-002]: ../milestones/mil-002-movie-page.md
[MIL-003]: ../milestones/mil-003-documentation.md
[RC-001]: ./reviews/rc-001-business-case.md
[RC-002]: ./reviews/rc-002-stakeholder-analysis.md
[RC-003]: ./reviews/rc-003-mil-001-foundation.md
[RC-004]: ./reviews/rc-004-mil-002-movie-page.md
[RC-005]: ./reviews/rc-005-mil-003-documentation.md
[RC-006]: ./reviews/rc-006-movie-page.md
[e8d24b2]: https://git.tirsystem.com/Tirsvad-Udemy-100-days-of-code/041-web-movie-ranking/commit/e8d24b2d32ce1ef8276123b0efa05b8a15e27d52
[5fe4ef0]: https://git.tirsystem.com/Tirsvad-Udemy-100-days-of-code/041-web-movie-ranking/commit/5fe4ef03673e70026a0667a2585f2d9decb6a2c2
View File
+22
View File
@@ -0,0 +1,22 @@
<!DOCTYPE html>
<html lang="en">
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1">
<title>The Best Movies According to Tirsvad</title>
<link rel="stylesheet" href="styles.css">
</head>
<body>
<main>
<h1>The Best Movies According to Tirsvad</h1>
<h2>My top 3 movies of all-time.</h2>
<hr>
<h3>13 element</h3>
<p>This is my favourite anime. I love the beautiful images.</p>
<h3>5th element</h3>
<p>Really cool sci-fi movie.</p>
<h3>Drive</h3>
<p>Super beautiful film. Really artistic.</p>
</main>
</body>
</html>
+33
View File
@@ -0,0 +1,33 @@
/* Readable typography for the Favorite Movie page. No external requests. */
:root {
color-scheme: light dark;
}
body {
margin: 0;
font-family: system-ui, -apple-system, "Segoe UI", Roboto, sans-serif;
line-height: 1.6;
}
main {
max-width: 40rem;
margin: 0 auto;
padding: 2rem 1rem;
}
h1 {
margin-bottom: 0.25rem;
}
h2 {
margin-top: 0;
font-weight: 400;
}
h3 {
margin-bottom: 0.25rem;
}
p {
margin-top: 0;
}
View File
+17
View File
@@ -0,0 +1,17 @@
# Manual test checklist: Favorite Movie page
Open `src/index.html` in a browser (see the README). Mark each item `Pass` or
`Fail` and note the date and browser.
Last run: 2026-10-24, built-in browser pane (Chromium), page served from a throwaway local static server so that `styles.css` loads (the pane shows `file://` pages as a static snapshot without the stylesheet). Items 1 to 5 and 8 were read from the DOM; item 6 from the resource list; item 7 at a 375 px wide viewport (scroll width equals client width).
| # | Check | Expected | Result |
| --- | --- | --- | --- |
| 1 | Exactly one `h1` | Text is "The Best Movies According to Tirsvad" | Pass |
| 2 | Exactly one `h2` | Text is "My top 3 movies of all-time." | Pass |
| 3 | Exactly one `hr` | A horizontal rule sits between the subtitle and the movies | Pass |
| 4 | Three `h3` in order | 13 element, 5th element, Drive | Pass |
| 5 | Each `h3` is directly followed by a `p` | Three paragraphs with the reasons | Pass |
| 6 | No external request | The network log shows only `index.html` and `styles.css` | Pass |
| 7 | Narrow window (375 px wide) | Text wraps and nothing scrolls sideways | Pass |
| 8 | Page title | The tab shows "The Best Movies According to Tirsvad" | Pass |