Skip to content

Conversation

@ignotus666
Copy link
Member

Short description of changes

Update es_ES app translation for 3.9.0

CHANGELOG: Updated Spanish translation of app for 3.9.0

Context: Fixes an issue?

Fixes #2693

Does this change need documentation? What needs to be documented and how?

No

Status of this Pull Request

Ready to be merged

What is missing until this pull request can be merged?

Checklist

  • I've verified that this Pull Request follows the general code principles
  • I tested my code and it does what I want
  • My code follows the style guide
  • I waited some time after this Pull Request was opened and all GitHub checks completed without errors.
  • I've filled all the content above

@ann0see ann0see added this to the Release 3.9.0 milestone Jul 4, 2022
@ann0see ann0see added the translation used by tools/create-translation-issues.sh label Jul 4, 2022
<location filename="../serverdlg.cpp" line="73"/>
<source>The registration of the server is renewed periodically to make sure that all servers in the connect dialog server list are actually available.</source>
<translation type="unfinished"></translation>
<translatorcomment>El registro del servidor se renueva periódicamente para asegurarse de que todos los servidores en la lista de servidores en la ventana de conexión están realmente disponibles.</translatorcomment>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you translate a comment and left the translation empty? Is this an error?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Argh! It's happened to me before... Typed into the wrong field. Fixed.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typed into the wrong field.

Just for info, what tool are you using?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QtLinguist

@pljones
Copy link
Collaborator

pljones commented Jul 6, 2022

All looks good to me - do we have someone else who can check the Spanish? (I can only "scan read" - it's close enough to English ;).)

Copy link
Member

@ann0see ann0see left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Translator listed in the src/util.cpp
  • Punctuation and spacing consistent (todo)
  • Signal words consistent ("ASIO", "Buffer") (todo)
  • No untranslated strings (grep unfinished -5 src/res/translation/translation_$TRANSLTION*.ts)
  • Only a single .ts file checked in (+ Installer translations)

@ann0see
Copy link
Member

ann0see commented Jul 6, 2022

Should still be squash merged ;-)

@ann0see ann0see requested a review from pljones July 14, 2022 17:32
Copy link
Collaborator

@pljones pljones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ann0see ann0see merged commit 602451f into jamulussoftware:master Jul 14, 2022
@pljones
Copy link
Collaborator

pljones commented Jul 14, 2022

The problem with Github squash-commit is that it doesn't squash merge - it squashes to a single, non-merge, commit on master.

@ann0see
Copy link
Member

ann0see commented Jul 14, 2022

Hmm. Yes. So you say it should be squashed locally and then merged?

@pljones
Copy link
Collaborator

pljones commented Jul 14, 2022

It needs a rebase -i, the squash/fix whatever on the commits, then merge. I was going to do it locally, yeah, the push directly.

It's one of Github's little annoyances. (Basically, you can't go editing the collaborator's repo, which is fair enough; you can pull their branch to a branch, then squash on that branch, though...)

@pljones pljones mentioned this pull request Jul 22, 2022
75 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

translation used by tools/create-translation-issues.sh

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update es_ES app translation for 3.9.0

3 participants