Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix 35806d61, which broke the selection of fluid ingredient variants. #136

Merged
merged 1 commit into from
May 23, 2024

Conversation

DaleStan
Copy link
Collaborator

In 35806d6, I changed recipe.links.ingredientGoods[i] to ingredient.goods, and that broke this dropdown in the ingredients column.
image

This changes it back and fixes the dropdown.

The various steam temperatures are most notable here.
@shpaass
Copy link
Owner

shpaass commented May 23, 2024

To clarify the bug, the steps to reproduce are as follows:

  1. Select any item that accepts a fluid ingredient with variants. For instance, Agar.
  2. Select a different variant.
  3. Observe that the fluid variant doesn't change.

The behavior of the fixed version is that the fluid variant changes.

@shpaass shpaass merged commit d2a36bf into shpaass:master May 23, 2024
1 check passed
@DaleStan DaleStan deleted the fix-fluid-ingredient-selection branch May 25, 2024 21:16
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