Skip to content

Add optional sowing density to sowing events - #87

Open
ANAMASGARD wants to merge 1 commit into
PecanProject:devfrom
ANAMASGARD:feat/55-sowing-density
Open

ANAMASGARD wants to merge 1 commit into
PecanProject:devfrom
ANAMASGARD:feat/55-sowing-density

Conversation

@ANAMASGARD

@ANAMASGARD ANAMASGARD commented Oct 18, 2026 •

Copy link
Copy Markdown

Fixes #55.

  • Adds an optional sowing density field (plants/m²) so users can record how densely each crop was sown, alongside or instead of seed weight.

  • The field uses the existing schema-driven form and accepts zero and fractional values. Both measurements remain optional, and existing records continue to work.

  • Regression tests cover validation, editing, persistence, and CSV/JSON exports. Local testing completed with 291 passing assertions.

Verification

  • 291 passing assertions, with one existing test skipped.
  • Browser checks confirmed validation, saving, reopening, editing, and CSV/JSON exports.
  • Package check completed with zero errors; existing warnings and note were reproduced on the unchanged baseline.

Screenshot

Reopened sowing event showing zero, blank, and fractional density values for different crops.
image

This branch has not been deployed

No deployments
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