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

Bug: Ingredient pill doesn't correctly parse next to semicolon #311

Closed
wlsnmrk opened this issue Dec 3, 2023 · 0 comments · Fixed by #317
Closed

Bug: Ingredient pill doesn't correctly parse next to semicolon #311

wlsnmrk opened this issue Dec 3, 2023 · 0 comments · Fixed by #317
Labels
bug Something isn't working

Comments

@wlsnmrk
Copy link

wlsnmrk commented Dec 3, 2023

Is this urgent?

No

What parts are affected

Frontend

What is the server version

0.4.19

What is the client version

0.4.19

What platform are you using

Web

What's the problem 🤔

The @ingredient_name syntax in the recipe description does not parse correctly if followed by a semicolon character (";"), and displays the literal string "@ingredient_name" instead of the ingredient pill.

Screenshot with a semicolon:
Screenshot 2023-12-03 150353

Screenshot of the same recipe, replacing the semicolon with a full stop:
Screenshot 2023-12-03 150440

Share your logs

No response

Share your configuration

No response

@wlsnmrk wlsnmrk added the bug Something isn't working label Dec 3, 2023
@TomBursch TomBursch moved this to In Progress in KitchenOwl Project Dec 6, 2023
@TomBursch TomBursch moved this from In Progress to Next Up in KitchenOwl Project Dec 6, 2023
@TomBursch TomBursch linked a pull request Dec 6, 2023 that will close this issue
@github-project-automation github-project-automation bot moved this from Next Up to Ready for Testing in KitchenOwl Project Dec 6, 2023
@TomBursch TomBursch moved this from Ready for Testing to Testing in KitchenOwl Project Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Testing
Development

Successfully merging a pull request may close this issue.

1 participant