Skip to content

Share compact homework review and public link controls - #309

Merged
alexeygrigorev merged 6 commits into
mainfrom
308-homework-review-lip-ui
Sep 26, 2026
Merged

alexeygrigorev merged 6 commits into
mainfrom
308-homework-review-lip-ui

Conversation

@alexeygrigorev

@alexeygrigorev alexeygrigorev commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

Part of #308. The issue remains open through both site adoptions.

Shared controls

  • Render one semantic, compact Review list with clear empty answers and Edit links, while keeping accepted and unsent draft snapshots separate. Long answers show a short preview with an expandable full answer.
  • Add an assignment-cap-aware Learning in Public link editor with optional blank state, add/remove controls, accessible field labels, and newline-backed answer storage compatible with stepper autosave.
  • Keep a visible textarea fallback so plain POST still works when JavaScript is unavailable.
  • Keep host styling in each consumer; the package owns structure and behavior.

Verification

  • uv run pytest -q tests/homework_steps/test_flow.py — 26 passed, including configured public links saved through the shared stepper and long-answer Review disclosure.
  • make check — passed before the last template/test change; CI reruns it on the current head.
  • Full local package suite on the preceding head: 2,221 passed, 3 skipped. CI reruns on the current head.

After package merge/release

@alexeygrigorev
alexeygrigorev force-pushed the 308-homework-review-lip-ui branch from c5eb950 to 0afb6a2 Compare September 26, 2026 19:59
@alexeygrigorev
alexeygrigorev marked this pull request as ready for review September 26, 2026 21:13
@alexeygrigorev
alexeygrigorev merged commit 2db0335 into main Sep 26, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Shared compact homework review and public-link controls

1 participant