Skip to content

Runestone: add termStartDate to eBookConfig#2804

Closed
ascholerChemeketa wants to merge 1 commit intoPreTeXtBook:masterfrom
ascholerChemeketa:term-start-date
Closed

Runestone: add termStartDate to eBookConfig#2804
ascholerChemeketa wants to merge 1 commit intoPreTeXtBook:masterfrom
ascholerChemeketa:term-start-date

Conversation

@ascholerChemeketa
Copy link
Copy Markdown
Contributor

Adds a single field to the eBookConfig to support a RS PR that will prevent student work done before the start of a class (e.g. when a student is repeating a course using the same book) from counting.

@ascholerChemeketa ascholerChemeketa marked this pull request as ready for review March 24, 2026 20:12
@ascholerChemeketa
Copy link
Copy Markdown
Contributor Author

RS PR: RunestoneInteractive/rs#1185

Copy link
Copy Markdown
Collaborator

@rbeezer rbeezer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested with the sample book using both the noparts (non-hosted) and academy (Runestone-hosted) publication files.

  • Non-hosted build: zero changes (as expected, the new line is only in the hosted branch)
  • Hosted build: eBookConfig.termStartDate appears on all 107 pages with the correct Jinja2 template variable

The new line follows the established pattern for all other eBookConfig fields.

🤖 Review by Claude Opus 4.6

@rbeezer
Copy link
Copy Markdown
Collaborator

rbeezer commented Mar 25, 2026

Yes, Claude was overkill for this, but now it "knows" how to review Runestone features. Before/after academy-hosted/generic-hosted tested manually.

@bnmnetp mentioned this feature in Drop-In yesterday, thanks for keeping the PreTeXt side current.

@rbeezer rbeezer closed this Mar 25, 2026
@ascholerChemeketa ascholerChemeketa deleted the term-start-date branch March 27, 2026 14:39
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.

2 participants