Money: New transaction writes an empty Notes property #1307

Closed
opened 2026-10-07 03:32:29 +00:00 by kayg · 2 comments
Owner

Steps

  1. Open a Budget account register.
  2. Add a transaction with a payee, account, category and amount.
  3. Leave Notes blank and save.
  4. Inspect the transaction row in the month's Budget.md file.

Expected

A transaction with no note has no notes property in its Markdown record.

Actual

The record contains - notes:: "" even though the User left Notes blank. This adds an empty field to every new transaction record created by the form.

Evidence

artifacts/money/money-empty-note-field.txt is the real-server E2E fixture output.

Severity: minor

## Steps 1. Open a Budget account register. 2. Add a transaction with a payee, account, category and amount. 3. Leave Notes blank and save. 4. Inspect the transaction row in the month's `Budget.md` file. ## Expected A transaction with no note has no `notes` property in its Markdown record. ## Actual The record contains `- notes:: ""` even though the User left Notes blank. This adds an empty field to every new transaction record created by the form. ## Evidence `artifacts/money/money-empty-note-field.txt` is the real-server E2E fixture output. Severity: minor
Author
Owner

Fixed on branch in 7eaa55bfe (fix(money): omit empty notes on new transactions). New transactions omit notes when the field is blank. Edits still send notes: "" so the User can clear an existing note. The two focused TransactionForm tests pass, and the unchanged real-server Markdown expectation passes.

Fixed on branch in `7eaa55bfe` (`fix(money): omit empty notes on new transactions`). New transactions omit `notes` when the field is blank. Edits still send `notes: ""` so the User can clear an existing note. The two focused `TransactionForm` tests pass, and the unchanged real-server Markdown expectation passes.
Author
Owner

Deployed to calternal.cloud in merge round 34 (d395b6e25), 2026-10-08 05:03 IST. Backup: /srv/calternal/backup-pre-m34-20261008-0132. Production healthy after 36 s; served build matches.

Deployed to calternal.cloud in merge round 34 (d395b6e25), 2026-10-08 05:03 IST. Backup: /srv/calternal/backup-pre-m34-20261008-0132. Production healthy after 36 s; served build matches.
kayg closed this issue 2026-10-07 23:54:35 +00:00
Commenting is not possible because the repository is archived.
No labels
No milestone
No project
No assignees
1 participant
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kayg/calternal-v0#1307
No description provided.