Skip to content

Recipe scaling#173

Merged
barneyb merged 10 commits intomasterfrom
recipe-scaling
Jan 18, 2026
Merged

Recipe scaling#173
barneyb merged 10 commits intomasterfrom
recipe-scaling

Conversation

@barneyb
Copy link
Member

@barneyb barneyb commented Jan 18, 2026

Several related changes:

  1. Remove nulls before diffing for recipe history. This was the source of the "can't delete subrecipe" over break.
  2. Better handling of spaces and commas when recognizing "complicated" raw values. I've made the same changes client-side, but haven't pushed them up yet.
  3. Bring auto-recognition (which happens sending to plan) inline with how the client does it. In particular, parse a quantity even if there is no ingredient.
  4. Don't treat the lack of a quantity to mean "one count".
  5. The big one: don't scale subrecipes onto the plan, or when computing recipe history after cooking it.
  6. The bulky one: consolidating has/noValue checks. All mechanical.

@barneyb barneyb merged commit 4640747 into master Jan 18, 2026
1 check passed
@barneyb barneyb deleted the recipe-scaling branch January 18, 2026 17:36
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