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 #464

Closed
wants to merge 2 commits into from

Conversation

kimloan2911
Copy link

@kimloan2911 kimloan2911 commented Aug 22, 2022

Description of the issue/feature this PR addresses:

Current behavior before PR:

  • Issues: 445
  • Reference of the module review meeting on August 17, 2022

Desired behavior after PR is merged:

  • Change Nhân viên bán hàng into Nhân viên Kinh doanh
  • Change Customer Invoices into Customer
  • Change Vendor Bills into Vendor
  • Video test:
simplescreenrecorder-2022-08-22_17.10.31.mp4

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

- Change `Customer Invoices` into `Customer`
- Change `Vendor Bills` into `Vendor`
@viinbot viinbot added 🚀 Running PR/Commit is done for everything and ready for manually test 🚀 Queue 🚀 Building and removed 🚀 Building 🚀 Running PR/Commit is done for everything and ready for manually test 🚀 Queue labels Aug 22, 2022
Copy link

@davidtranhp davidtranhp left a comment

Choose a reason for hiding this comment

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

@royleviindoo @tranngocson1996 Cái này sửa xml thì cần làm phía bên Odoo hoặc phải tạo module bên nhà mình để xử chứ nhỉ? Nếu xử ở đây thì sẽ không được forward lên các phiên bản sau

@tranngocson1996
Copy link

@davidtranhp dạ vâng anh ạ, em cũng đã trao đổi với Loan trước đó rồi ạ
@kimloan2911 những phần dịch liên quan đến code của Odoo em phải sửa bên Odoo và giải thích rõ ràng cho họ tại sao mình lại sửa như vậy nhé

@kimloan2911
Copy link
Author

kimloan2911 commented Aug 23, 2022

@tranngocson1996 Dạ vâng, em làm PR lên Odoo rồi ạ

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