feat(app): implement objective summary step in contribution flow by Carter907 · Pull Request #166 · bluelearn-org/bluelearn

Carter907 · 2026-07-21T01:59:45Z

Summary

Curriculum Receipt (Submit Step): Implemented the final summary view as a "receipt-style" vertical timeline. It clearly lists out the curated targets and their ordered prerequisites in the exact sequence the curator organized them.
Subject Tagging: Expanded the Objective Contribution state and the ObjectiveDetails step to include a subjects Combobox, allowing curators to tag their objective with multiple subjects (which are displayed on the summary page).
Frictionless Form Validation UX: Overhauled the @stepperize/react validation pattern in both the ObjectiveDetails and Order Guides steps. Instead of silently locking the "Next" button when a form is incomplete, the UI now intercepts the click, shows a clear error toast, and dynamically applies Shadcn's destructive styling (border-destructive ring-destructive) to any missing fields.
Strict Sequence Verification: Added validation logic to the Order Guides step that explicitly blocks progression until the curator has visited every single target guide's prerequisite graph. Any unvisited targets are explicitly highlighted in red if they try to skip ahead.

closes #157
closes #164

Type of change

Verification

pnpm -r typecheck passes
pnpm -r build passes
Manually verified in a browser (for UI / behavior changes)
Followed the app layout conventions
No new dependencies, or new dependencies are justified and AGPL-compatible
Change does not violate any non-negotiable principle
Documentation only, no changes to the codebase, so no tests required

Signed-off-by: Carter <speers.carter@gmail.com>

…tion Signed-off-by: Carter <speers.carter@gmail.com>

…etails Signed-off-by: Carter <speers.carter@gmail.com>

Signed-off-by: Carter <speers.carter@gmail.com>

andyrodrigues30

See annotated screenshot.

Also remove the submit button from the bottom and keep it at the top next to the submit button like the other steps.

Signed-off-by: Carter <speers.carter@gmail.com>

andyrodrigues30

See comments

andyrodrigues30 · 2026-07-23T20:57:32Z

+        publishText="Submit"
      />

      <Separator className="mb-8 bg-border" />


Remove this (it already exists in the StepperActionHeader)

andyrodrigues30 · 2026-07-23T21:04:20Z

      <Separator className="mb-8 bg-border" />
+
+      <div className="mx-auto mt-8 flex w-full max-w-3xl flex-col gap-12">
+        <Card className="rounded-md bg-background shadow-none">


Display this card of "Objective" information like this the following objective card (that way we know what the actual objective card will look like. This card should take up the full width.

Add the tags under the username and created at date.

You don't need the "Not Started" status.
For the stats in the footer - it would be 3 stats:

Duration (use M for minutes)

Sub-objectives (instead of target guides

Guides (this is the total number of guides)

andyrodrigues30 · 2026-07-23T21:05:03Z

+
+        <div className="space-y-6">
+          <h3 className="px-1 text-lg font-semibold tracking-tight">
+            Curriculum Paths


Change to "Sub-objectives"

andyrodrigues30 · 2026-07-23T21:07:23Z

+                    key={idx}
+                    className="flex flex-col gap-2 md:flex-row md:items-center md:gap-4"
+                  >
+                    <div className="flex h-8 w-8 shrink-0 items-center justify-center rounded-full border border-badge-border bg-badge font-mono text-xs font-semibold text-badge-foreground">


Increase the spacing between the order number and the card see below (have a look how it is implemented on objectives.index.tsx

andyrodrigues30 · 2026-07-23T21:08:04Z

+                                  </span>
+                                  {getGuideDuration(stepSlug) > 0 && (
+                                    <span className="shrink-0 font-mono text-[10px] text-muted-foreground uppercase">
+                                      {getGuideDuration(stepSlug)} mins


change "mins" to "m"

Signed-off-by: Carter <speers.carter@gmail.com>

- Persist objective contribution form state in sessionStorage to survive page reloads. - Decouple PreviewObjectiveCard from ObjectiveCard to safely refine preview styling. - Center arrows and tighten featured guide graphs on wide screens. - Append summaries and subject tags to subobjective target cards. - Synchronize featured guide graph with user's explicitly selected featured guide. - Format duration as 'min' instead of 'M'. Signed-off-by: Carter <speers.carter@gmail.com>

Carter907 · 2026-07-25T15:35:42Z

Here is the updated Objective Card on the preview page:

Carter907 · 2026-07-25T15:36:17Z

here is the sub-objectives:

Replace static mock data with backend service calls - Implemented add and removal of objective targets to objective-revision.service.ts. - Fetch walkthrough edges dynamically from /:slug/walkthrough API. - Placeholder Preview page based on PR bluelearn-org#166 - Added 'Save Draft' button for drafting LOs. Signed-off-by: Carter <speers.carter@gmail.com>

tonylam0 · 2026-07-31T00:30:30Z

Closing this PR because it's changes were included in PR 207.

Carter907 added 4 commits July 20, 2026 20:39

feat(app): implement objective summary step and fix combobox TS errors

f7426b4

Signed-off-by: Carter <speers.carter@gmail.com>

feat(app): add subjects to objective flow and enforce sequence valida…

4bed16a

…tion Signed-off-by: Carter <speers.carter@gmail.com>

feat(app): add validation toast and field highlighting to objective d…

a955a04

…etails Signed-off-by: Carter <speers.carter@gmail.com>

feat(app): highlight unvisited targets in order guides step

9f4e67b

Signed-off-by: Carter <speers.carter@gmail.com>

Carter907 requested review from andyrodrigues30 and tonylam0 as code owners July 21, 2026 01:59

Merge branch 'main' into feat/objective-summary-step

099b597

Carter907 changed the title ~~Feat/objective summary step~~ feat(app): objective summary step Jul 21, 2026

Carter907 changed the title ~~feat(app): objective summary step~~ feat(app): implement objective summary step in contribution flow Jul 21, 2026

Carter907 self-assigned this Jul 21, 2026

Carter907 added the area:frontend Tasks related to frontend (app) label Jul 21, 2026

andyrodrigues30 requested changes Jul 22, 2026

View reviewed changes

feat(app): merge objective contribution flow

8e8540a

Signed-off-by: Carter <speers.carter@gmail.com>

Carter907 requested a review from andyrodrigues30 July 23, 2026 06:52

andyrodrigues30 requested changes Jul 23, 2026

View reviewed changes

style(app): polish objective preview UI

b08bd39

Signed-off-by: Carter <speers.carter@gmail.com>

Carter907 force-pushed the feat/objective-summary-step branch from 78485e8 to b08bd39 Compare July 24, 2026 06:27

Carter907 and others added 2 commits July 24, 2026 04:25

Merge branch 'bluelearn-org:main' into feat/objective-summary-step

9deb100

Carter907 requested a review from andyrodrigues30 July 25, 2026 15:31

Merge branch 'bluelearn-org:main' into feat/objective-summary-step

f37037f

Carter907 mentioned this pull request Jul 26, 2026

feat(app): integrate API for objective drafting and publishing #207

Merged

13 tasks

tonylam0 closed this Jul 31, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(app): implement objective summary step in contribution flow - #166

feat(app): implement objective summary step in contribution flow#166
Carter907 wants to merge 10 commits into
bluelearn-org:mainfrom
Carter907:feat/objective-summary-step

Carter907 commented Jul 21, 2026 •

edited

Loading

Uh oh!

andyrodrigues30 left a comment

Uh oh!

andyrodrigues30 left a comment

Uh oh!

andyrodrigues30 Jul 23, 2026

Uh oh!

andyrodrigues30 Jul 23, 2026

Uh oh!

andyrodrigues30 Jul 23, 2026

Uh oh!

andyrodrigues30 Jul 23, 2026

Uh oh!

andyrodrigues30 Jul 23, 2026

Uh oh!

Carter907 commented Jul 25, 2026

Uh oh!

Carter907 commented Jul 25, 2026

Uh oh!

tonylam0 commented Jul 31, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

Carter907 commented Jul 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Type of change

Verification

Uh oh!

andyrodrigues30 left a comment

Choose a reason for hiding this comment

Uh oh!

andyrodrigues30 left a comment

Choose a reason for hiding this comment

Uh oh!

andyrodrigues30 Jul 23, 2026

Choose a reason for hiding this comment

Uh oh!

andyrodrigues30 Jul 23, 2026

Choose a reason for hiding this comment

Uh oh!

andyrodrigues30 Jul 23, 2026

Choose a reason for hiding this comment

Uh oh!

andyrodrigues30 Jul 23, 2026

Choose a reason for hiding this comment

Uh oh!

andyrodrigues30 Jul 23, 2026

Choose a reason for hiding this comment

Uh oh!

Carter907 commented Jul 25, 2026

Uh oh!

Carter907 commented Jul 25, 2026

Uh oh!

tonylam0 commented Jul 31, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Carter907 commented Jul 21, 2026 •

edited

Loading