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

Pluralization of ingredients with long names #106

Closed
johnhutch opened this issue Jun 2, 2017 · 6 comments
Closed

Pluralization of ingredients with long names #106

johnhutch opened this issue Jun 2, 2017 · 6 comments
Assignees
Labels

Comments

@johnhutch
Copy link
Collaborator

E.g., chopped.

@johnhutch johnhutch added the bug label Jun 2, 2017
@johnhutch johnhutch self-assigned this Jun 9, 2017
@johnhutch
Copy link
Collaborator Author

Covered in #112

@johnhutch johnhutch removed their assignment Jun 22, 2017
@johnhutch
Copy link
Collaborator Author

Please review in conjunction with #112

@jon-athan-hall
Copy link
Contributor

See code review for #112.

@johnhutch johnhutch removed their assignment Jul 6, 2017
@jon-athan-hall
Copy link
Contributor

Looks to be working on PROD. I included "4 tbsps chopped onion" and it saved correctly. Hell of a bug description by the way! hahaha, NOT.

Ready for second QA by original developer.

@johnhutch
Copy link
Collaborator Author

Yeah, crud, that IS a dumb bug description. I think it was a mispaste? The e.g., shoudl have had the whole ingredients block. Woops!

@johnhutch
Copy link
Collaborator Author

Verified after adding a new recipe with the following ingredient set:

5 tbsp soy sauce
4 tbsp mild or hot chili powder
3 tbsp five-spice powder
2 tbsp light muscovado sugar
2 cloves garlic, finely chopped
4 cm piece of fresh ginger, peeled and finely chopped
6 1/2 lbs boneless pork shoulder, rind removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants