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] website_blog: improve vietnamese translation #504

Merged
merged 2 commits into from
Sep 6, 2022

Conversation

huy97852
Copy link

@huy97852 huy97852 commented Sep 6, 2022

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

*small changes
*add missing translation
*change field name to CamelCase format

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

@huy97852 huy97852 changed the title [I18N] website_blog: improve vietnamese translation [I18N][15.0] website_blog: improve vietnamese translation Sep 6, 2022
@viinbot viinbot added 🚀 Running PR/Commit is done for everything and ready for manually test and removed 🚀 Building labels Sep 6, 2022
@@ -46,7 +46,7 @@ msgstr "<?xml version=\"1.0\" encoding=\"utf-8\"?>"
#. module: website_blog
#: model_terms:ir.ui.view,arch_db:website_blog.blog_post_short
msgid "' page header."
msgstr "'."
msgstr "'tiêu đề trang."

Choose a reason for hiding this comment

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

@huy97852 Nó đang đúng lại sửa thành sai. Cần phải xem kỹ code cũng như mở giao diện phần mềm ra để đối chiếu.

Suggested change
msgstr "'tiêu đề trang."
msgstr "'."

Đây là code ở xml:

<t t-set="oe_structure_blog_single_header_description">Edit the '<t t-esc="blog.name"/>' page header.</t>

Và thuật ngữ được tách thành 2 phần:

  1. msgid "Edit the '"
    msgstr "Sửa tiêu đề trang '"
  2. msgid "' page header."
    msgstr "'."

Khi ghép chúng nó lại với nhau thì sẽ đúng, kết quả sẽ có dạng Sửa tiêu đề trang 'blog ABCD'. thay vì Sửa 'blog ABCD' tiêu đề trang.

@Viindoo/product @Viindoo/is @Viindoo/project Anh em dịch cần lưu ý case này nhé.
@leotranviindoo @royleviindoo @phamgiang2510 Các chú lưu ý với anh em chỗ này nhé. Vì anh gặp lỗi này của anh em vài lần rồi.

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 ạ , em sửa lại ngay

Choose a reason for hiding this comment

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

vâng anh.

@viinbot viinbot added 🚀 Queue 🚀 Building 🚀 Running PR/Commit is done for everything and ready for manually test and removed 🚀 Running PR/Commit is done for everything and ready for manually test 🚀 Queue 🚀 Building labels Sep 6, 2022
@davidtranhp davidtranhp merged commit 69e3b58 into Viindoo:15.0 Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 Running PR/Commit is done for everything and ready for manually test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants