Skip to content

Fix notes never rendering in text output - #23

Merged
itspriddle merged 1 commit into
mainfrom
bugfix/notes
Sep 25, 2026
Merged

itspriddle merged 1 commit into
mainfrom
bugfix/notes

Conversation

@itspriddle

@itspriddle itspriddle commented Sep 25, 2026 •

Copy link
Copy Markdown
Owner

Relates to #21

The default event template never referenced {{notes}}, so show-notes = true and --hide notes had no effect on text output. Notes only appeared in JSON.

This adds a Notes section to the default template, after Attendees and before Recurs. It's still off by default (show-notes = false).

  • Multi-line notes are indented under the Notes: label via a new {{{indentedNotes}}} template variable. Both LF and CRLF line endings work, and blank lines get no trailing whitespace.
  • Notes are rendered unescaped, so <b> is not turned into &lt;b&gt;.
  • truncate-notes still applies.

Example with show-notes = true:

    Notes:
      <b>Booked by</b>
      Josh Priddle
      josh@josh.fail

Not in this PR: the --hide recurrence / show-recurrence toggle and stripping HTML from notes.

The default event template never referenced notes, so show-notes and
--hide notes had no effect.
@itspriddle
itspriddle merged commit 589824b into main Sep 25, 2026
1 check passed
@itspriddle
itspriddle deleted the bugfix/notes branch September 25, 2026 15:35
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.

1 participant