Add chili sauce to Umami

Here's the image now cropped, tweaked and compressed at 768px wide.

This patch applies cleanly and looks great. Content looks ready to go.

Marking as Needs work because the only issue I've spotted is that the image should be added to the licence.txt file.

Adding a screenshot so others can review easily.

Hi, I'm in the sprint room, and I am going to review this patch.

I applied the patch pre-install and everything looks good. Attaching images before and after the patch.

#2952059: Add Crema Catalana to Umami was added.

Rerolled.

Tested patch being applied after reroll. This looks good to me.

Marking as reviewed and tested.

Discussed the recipe title a bit with @lauriii and @eli-t (in separate chats). The issue is what does "Mwmbwldraig" mean? It is explained in the text but it does induce a wtf. Also it was pointed out that chillies are not from Wales and none of the other recipes has this type of name. They all are a well recognised name of dish.

Recommend changing name to "Fiery chili sauce"

Patch applied successfully with some warnings. Attaching screenshot for refrence.

The problem mentioned in #27 is only in the image, and we shouldn't worry about that.

I did have some small questions about the text, but I haven't read the writing style issues for umami yet, so feel free to ignore those. They were the only nits I had to pick, otherwise this looks great.

1. +++ b/core/profiles/demo_umami/modules/demo_umami_content/default_content/recipe_instructions/chili-sauce-umami.html
   @@ -0,0 +1,14 @@
   +  <li>Meanwhile, chop the bell peppers to 1 inch pieces. Add the bell peppers and tomatoes to a roasting dish, and rub with lemon juice, olive oil and black pepper. Transfer to the oven and roast for 30 minutes.</li>
   

   I'm not a native speaker, but this is the first time that I've seen a recipe say to rub with lemon juice. Is sprinkle the same?

2. +++ b/core/profiles/demo_umami/modules/demo_umami_content/default_content/recipe_instructions/chili-sauce-umami.html
   @@ -0,0 +1,14 @@
   +  <li>Once the tomatoes and bell peppers have finished roasting, add them to the pan. Simmer for at least 10 minutes, breaking up as you go. Once broken up, mix in the sugar.</li>
   

   I'm not sure about the language here either.

   Simmer for at least 10 minutes, breaking up as you go. Once broken up, mix in the sugar.
   I would replace that with something like:
   Simmer for at least 10 minutes. When the vegetables are broken up, mix in the sugar.

   That reads easier for me.

Yes, that makes it clearer what action is needed, I prefer that.

Patch 36 looks good. Tests seem to pass. Waiting for 1 more person to confirm before RTBC.

There was a testbot snafu over the weekend.

@Mukeysh Thank you for reviewing this issue!

The automated testing infrastructure tells us whether the patch applies, so we do not need people to review that. It is also not sufficient criteria for the issue to be marked "Reviewed and Tested by the Community".

What we do need people to review is whether the issue has a correct scope, whether it passes the core gates, whether the solution completely fixes the problem without introducing other problems, and whether it's the best solution we can come up with. See the patch review guide for more information.

When you do post a review, be sure to describe what you reviewed and how. This helps other reviewers understand why you considered the issue RTBC (and is considered for issue credit).

Posting screenshots of your codebase is not helpful, since the automated testing infrastructure tells us whether the patch applies correctly.

So, I've removed your credit for this issue. In the future, you can get credit for issues by reading the issue to understand its purpose, and posting your review or testing of that purpose.

Committed and pushed 6d4cd5d404 to 8.6.x and c126800aaa to 8.5.x. Thanks!

@alexpott thanks for your suggestions. I will take care of this in future.