From 2b9928983f72b89fc22a47a190e25fc64c3610c3 Mon Sep 17 00:00:00 2001
From: Daniel Wagner-Hall <dawagner@gmail.com>
Date: Mon, 13 Apr 2026 16:23:24 +0100
Subject: [PATCH 1/2] Remind readers to check they've met learning objectives

This shows up on every day-plan block which has learning objectives (but
not tasks).

Currently only supported for local and readme blocks, but I think that's
all of the relevant ones.

Doesn't show up on non-day-plan pages (e.g. prep).
---
 common-theme/layouts/partials/block/data.html        | 1 +
 common-theme/layouts/partials/block/local.html       | 3 +++
 common-theme/layouts/partials/block/readme.html      | 3 +++
 common-theme/layouts/partials/review-objectives.html | 4 ++++
 4 files changed, 11 insertions(+)
 create mode 100644 common-theme/layouts/partials/review-objectives.html

diff --git a/common-theme/layouts/partials/block/data.html b/common-theme/layouts/partials/block/data.html
index 8c02b0164..5b3fdcb42 100644
--- a/common-theme/layouts/partials/block/data.html
+++ b/common-theme/layouts/partials/block/data.html
@@ -29,6 +29,7 @@
 {{ .Scratch.SetInMap "blockData" "hasBlocks" .hasBlocks| default "" }}
 {{/* Override any block time */}}
 {{ .Scratch.SetInMap "blockData" "time" .time | default "" }}
+{{ .Scratch.SetInMap "blockData" "pageLayout" $pageRef.Page.Params.layout}}
 
 {{/* Now let's do our headers */}}
 {{ $headers := partial "github-auth.html" }}
diff --git a/common-theme/layouts/partials/block/local.html b/common-theme/layouts/partials/block/local.html
index 51eab29d1..374e67af0 100644
--- a/common-theme/layouts/partials/block/local.html
+++ b/common-theme/layouts/partials/block/local.html
@@ -46,5 +46,8 @@ <h3>🗂️ Options</h3>
         {{ end }}
       </section>
     {{ end }}
+    {{ if isset $block.Params "objectives" }}
+    {{ partial "review-objectives.html" $blockData }}
+    {{ end }}
   </div>
 </section>
diff --git a/common-theme/layouts/partials/block/readme.html b/common-theme/layouts/partials/block/readme.html
index 68952af58..e7c91d50b 100644
--- a/common-theme/layouts/partials/block/readme.html
+++ b/common-theme/layouts/partials/block/readme.html
@@ -63,5 +63,8 @@ <h3>🗂️ Related</h3>
         {{ end }}
       </section>
     {{ end }}
+    {{ if gt (len $extractedObjectives) 0 }}
+    {{ partial "review-objectives.html" $blockData }}
+    {{ end }}
   </section>
 {{ end }}
diff --git a/common-theme/layouts/partials/review-objectives.html b/common-theme/layouts/partials/review-objectives.html
new file mode 100644
index 000000000..bb4dcacca
--- /dev/null
+++ b/common-theme/layouts/partials/review-objectives.html
@@ -0,0 +1,4 @@
+{{ if eq .pageLayout "day-plan" }}
+<h3>Review</h3>
+Look back over the objectives of this activity - check you've met them all. If you haven't, make sure you have a plan for how to achieve them.
+{{ end }}

From ec7eaec115fd1fdd882f9e8ee441c88fedf2c06f Mon Sep 17 00:00:00 2001
From: Daniel Wagner-Hall <dawagner@gmail.com>
Date: Tue, 14 Apr 2026 16:47:31 +0100
Subject: [PATCH 2/2] Add wording based on feedback

---
 common-theme/layouts/partials/review-objectives.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/common-theme/layouts/partials/review-objectives.html b/common-theme/layouts/partials/review-objectives.html
index bb4dcacca..df4f1f5e0 100644
--- a/common-theme/layouts/partials/review-objectives.html
+++ b/common-theme/layouts/partials/review-objectives.html
@@ -1,4 +1,4 @@
 {{ if eq .pageLayout "day-plan" }}
 <h3>Review</h3>
-Look back over the objectives of this activity - check you've met them all. If you haven't, make sure you have a plan for how to achieve them.
+Look back over the objectives of this activity - check you've met them all. If you haven't, make sure you have a plan for how to achieve them - maybe checking in with a volunteer or a fellow trainee could help?
 {{ end }}