Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Tgalog translations for security and validation messages. #26106

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

Closed
wants to merge 11 commits into from
Closed

Tgalog translations for security and validation messages. #26106

wants to merge 11 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Feb 9, 2018

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? no
Fixed tickets none
License MIT
Doc PR symfony/symfony-docs#...

ergiegonzaga added 4 commits February 9, 2018 16:52
Translated the document in Tagalog Language.
Update secuity_tagalog translations.ar.xlf
…tors_tagalog-translations.af.xlf

Update validators_tagalog translations.af.xlf
@javiereguiluz
Copy link
Member

javiereguiluz commented Feb 9, 2018

@ergiegonzaga thanks for contributing this translation! However, there are some issues that we need to fix: instead of creating a new translation for Tagalog, you modified the translation for Arabic. This is how we can fix it:

  1. Copy the src/Symfony/Component/Security/Core/Resources/translations/security.en.xlf file into a new file called src/Symfony/Component/Security/Core/Resources/translations/security.tl.xlf (tl is the code for the Tagalog language).
  2. Make the translation in the new security.tl.xlf file ... you must keep the original English contents in <source> ... </source> and you must write your translations in the <target> ... </target> nodes.
  3. Do the same for validators translation: copy src/Symfony/Component/Validator/Resources/translations/validators.en.xlf into a new file called src/Symfony/Component/Validator/Resources/translations/validators.tl.xlf, keep the <source> ... </source> contents unchanged and add your translations in <target> ... </target>.
  4. Revert the changes you made in the src/Symfony/Component/Security/Core/Resources/translations/security.ar.xlf and src/Symfony/Component/Validator/Resources/translations/validators.af.xlf files.

Thank you!

@stof
Copy link
Member

stof commented Feb 9, 2018

On a side note, you should generally not use your master branch to open pull requests. This will cause your master branch to be out of sync with the upstream one, making it harder for you on your next contribution.

@ghost
Copy link
Author

ghost commented Feb 9, 2018 via email

@stof
Copy link
Member

stof commented Feb 9, 2018

@ergiegonzaga the simpler solution is to start a new PR using a feature branch created from the Symfony master (so starting clean).

@javiereguiluz
Copy link
Member

The easiest way to revert is to:

  1. Copy these contents -> https://raw.githubusercontent.com/symfony/symfony/master/src/Symfony/Component/Security/Core/Resources/translations/security.ar.xlf and paste them into src/Symfony/Component/Security/Core/Resources/translations/security.ar.xlf
  2. Copy these contents -> https://raw.githubusercontent.com/symfony/symfony/master/src/Symfony/Component/Validator/Resources/translations/validators.af.xlf into src/Symfony/Component/Validator/Resources/translations/validators.af.xlf
  3. Commit those changes and push them.

Ergie Gonzaga added 2 commits February 9, 2018 20:07
Translations for Tagalog Dialect
@ghost
Copy link
Author

ghost commented Feb 9, 2018 via email

Ergie Gonzaga added 5 commits February 9, 2018 20:14
Revert changes I've made earlier
Tagalog translation
Revert Changes I've made earlier.
@ghost
Copy link
Author

ghost commented Feb 9, 2018 via email

@xabbuh
Copy link
Member

xabbuh commented Feb 9, 2018

closing in favour of #26114

@xabbuh xabbuh closed this Feb 9, 2018
@ghost
Copy link
Author

ghost commented Feb 9, 2018 via email

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

Successfully merging this pull request may close these issues.

4 participants