Skip to content
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

[I18N][15.0] account*: Update vi.po translation file #511

Open
wants to merge 1 commit into
base: 15.0
Choose a base branch
from

Conversation

kimloan2911
Copy link

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

  • Change to the CamelCase format
  • Video: This PR only edit translation file, so video recording is not necessary

--
I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

@@ -1389,7 +1389,7 @@ msgstr "Hướng dẫn thanh toán hoàn tiền"
#: model:ir.model,name:payment.model_payment_token
#: model:ir.model.fields,field_description:payment.field_payment_transaction__token_id
msgid "Payment Token"
msgstr "Mã thanh toán"
msgstr "Mã Thanh toán"

Choose a reason for hiding this comment

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

@kimloan2911 Token thì không nên dịch thành Mã. Nên dịch nó thành Mã bảo mật hoặc để nguyên là token. Chúng ta thấy là các ngân hàng VN giờ cũng dùng thuật ngữ Token khá nhiều nên đại đa số dân chúng sẽ dần quen với token. Nên có lẽ không cần dịch token nữa

Suggested change
msgstr " Thanh toán"
msgstr "Token Thanh toán"

CC @leotranviindoo @royleviindoo @phamgiang2510 @Viindoo/is @Viindoo/product @Viindoo/project

Copy link
Author

Choose a reason for hiding this comment

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

Dạ vâng anh, để em sửa lại ạ.

addons/payment/i18n/vi.po Outdated Show resolved Hide resolved
addons/payment/i18n/vi.po Outdated Show resolved Hide resolved
addons/account_edi/i18n/vi.po Outdated Show resolved Hide resolved
addons/account/i18n/vi.po Outdated Show resolved Hide resolved
addons/account/i18n/vi.po Outdated Show resolved Hide resolved
addons/account/i18n/vi.po Show resolved Hide resolved
addons/account/i18n/vi.po Outdated Show resolved Hide resolved
@davidtranhp
Copy link

Nhớ rebase và xử lý conflict trước nhé @kimloan2911

@kimloan2911
Copy link
Author

Nhớ rebase và xử lý conflict trước nhé @kimloan2911

Dạ vâng anh.

- Add missing translation
- Change to the CamelCase format
@viinbot viinbot added 🚀 Queue 🚀 Building ⚠️ Failed and removed 🚀 Running PR/Commit is done for everything and ready for manually test 🚀 Queue ⚠️ Failed labels Sep 14, 2022
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.

3 participants