Support from Acquia helps fund testing for Drupal Acquia logo

Comments

shaal created an issue. See original summary.

shaal’s picture

Version: 8.7.x-dev » 8.8.x-dev

Drupal 8.7.0-alpha1 will be released the week of March 11, 2019, which means new developments and disruptive changes should now be targeted against the 8.8.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

shaal’s picture

The patch includes CSV files of recipes that have a translation

smaz’s picture

Status: Needs review » Reviewed & tested by the community

Patch applies cleanly, Umami still installs ok, CSV file in the correct format. This looks fine to me!

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Needs work

#3028791: Umami Recipe Spanish Translation: Super easy vegetarian pasta bake is fixed as well. Since we don't have the crediting situation figured out here anyway, we have an opportunity to include that too.

kjay credited Eli-T.

kjay credited estoyausente.

kjay credited isholgueras.

kjay credited rteijeiro.

kjay credited saesa.

kjay’s picture

kjay credited rjfg.

kjay’s picture

Adding credit for translation work done on the Vegan Chocolate recipe.

shaal’s picture

shaal’s picture

rteijeiro’s picture

Status: Needs review » Reviewed & tested by the community

Patch applies well and recipe and instructions files are created properly. If there is anything else I must check, just let me know.

Gábor Hojtsy’s picture

Title: CSV files for Umami's recipes in Spanish » Spanish translations of Umami demo recipes
Status: Reviewed & tested by the community » Needs work

I looked at the markup, since I don't know Spanish :) Markup looks fine. Spanish was reviewed and provided by folks @kjay already credited. This needs fixing though:

+++ b/core/profiles/demo_umami/modules/demo_umami_content/default_content/languages/es/recipe_instructions/veggie-pasta-bake-umami.html
@@ -0,0 +1,8 @@
+</ol>
\ No newline at end of file

Certainly needs a newline here :)

shaal’s picture

Status: Needs work » Needs review
FileSize
12.54 KB
598 bytes

Added the missing newline at the end of veggie-pasta-bake-umami.html file.

shaal’s picture

Status: Needs review » Reviewed & tested by the community

A minor fix, self RTBC'ing :)

  • Gábor Hojtsy committed 9006714 on 8.8.x
    Issue #3038309 by shaal, rteijeiro, kjay, Eli-T, smaz, estoyausente,...

  • Gábor Hojtsy committed 6a3e6fa on 8.7.x
    Issue #3038309 by shaal, rteijeiro, kjay, Eli-T, smaz, estoyausente,...
Gábor Hojtsy’s picture

Version: 8.8.x-dev » 8.7.x-dev
Status: Reviewed & tested by the community » Fixed

Superb, thanks, committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.