Conversation
|
I added a lot of corrections to the spanish translation at Crowdin that aren't present on the commits. Can you review them? |
Once they're approved in Crowdin they'll show in the commits. |
|
@miguerubsk You can try editing the strings in Crowdin. Unfortunately, there’s no easy way to apply all your suggestions at once. I might need to ‘unapprove’ the current translations so you can edit them—let’s see if this works first. |
|
@svenvg93 I'm aware I edited a lot of strings in Crowdin, I left comments in all of them to try to ease the work, let me know if can help in any way |
I saw that 😅 and we really appreciate the work. Unfortunately, we can’t easily apply the suggestions from the comments without manually copying and pasting them all. We’re trying to find a simpler solution. Can you edit the strings where you left the comments, or are they locked? 
|
|
I thought i edited the strings, let me check it, it's just that string or every string? |
I think every string where you commented. Let me know if it works. Otherwise I’ll try to figure out how to do it |
I can see this in the translations tab, maybe I didn't do it right? 
|
|
Hmm when I open the Spanish translations all I can see is comment not the "new" translation in the string 
Crowdin seems more complicated then we thought 😆 |
|
That's strange, is there anything I can do? |
|
I will need to dig deeper in all the ints and outs of crowdin. |
Yes, I can go that way, I can click and edit translations and then save them, after that Crowdin asks if I want to notify the edit, I think those are the messages, but I don't understand why you can't see the translations Maybe it's because they already had approved translations? |
|
Ahh that might have been the reason. I removed the Approval for the ones that had a comment and now I can see the changes. Seems more complicated then it need to be 😅 . There are a few QA isseu that crowdin marked. If you can check them and if needed "ignore" them with the little X. I will approve them again so Crowdin can commit them. Sorry for all the trouble! |
lang/es_ES/api_tokens.php
Outdated
| 'expires_at_helper_text' => 'Dejar en blanco si no desea una fecha de caducidad', | ||
| 'last_used_at' => 'Último usado en', | ||
| 'expires_at_helper_text' => 'Deja este campo en blanco si no deseas una fecha de caducidad', | ||
| 'last_used_at' => 'Usado por última vez él', |
There was a problem hiding this comment.
"él" is not correct in this context, the correct form is "el" because is not acting as a subject, I can edit this on crowdin when I'm back home
lang/es_ES/dashboard.php
Outdated
| 'title' => 'Panel de Control', | ||
| 'no_speedtests_scheduled' => 'No hay pruebas de velocidad programadas.', | ||
| 'next_speedtest_at' => 'Siguiente prueba de velocidad en', | ||
| 'next_speedtest_at' => 'Siguiente prueba de velocidad él', |
| 'export_all_results_description' => 'Exportará todas las columnas para todos los resultados.', | ||
| 'export_completed' => 'Exportación completada, :count :rows exportadas.', | ||
| 'failed_export' => ':count :rows falló al exportar.', | ||
| 'failed_export' => ':count :ros exportación fallida.', |
There was a problem hiding this comment.
I think ":ros" is a typo that will break the code, will edit later if you confirm is wrong
lang/es_ES/results.php
Outdated
| 'created_from' => 'Creado a partir de', | ||
| 'only_healthy_speedtests' => 'Solo pruebas de velocidad saludables', | ||
| 'only_unhealthy_speedtests' => 'Sólo pruebas de velocidad no saludables', | ||
| 'only_manual_speedtests' => 'Solo pruebas de velocidad manual', |
There was a problem hiding this comment.
sorry, I probably messed up this, should be "manuales", will correct at crowdin
miguerubsk
left a comment
miguerubsk
left a comment
There was a problem hiding this comment.
there were some mistakes, I think some are crowdin's fault but others are my fault, how can I edit them on crowdin to avoid the problems of they not apearing?
Just report the issue, as you just did for three others. I can take it from there. |
This was just a summary, I commented the errors I saw and tried to correct them on Crowdin again |
3 participants
No description provided.