Re: [Openerp-community] [Merge] lp:~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn into lp:openobject-addons/extra-trunk

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: [Openerp-community] [Merge] lp:~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn into lp:openobject-addons/extra-trunk

Numérigraphe
"2. For language localization, what should be the best practice (...)?"
Ideally you should have the data in English in the data files, and a translation into your language. That will let interested contributors translate it to their own language.
But often, the data for l10n projects is only available in the native language of the project, so it's accepted that you provide that instead of English.
Lionel Sausin.
--
https://code.launchpad.net/~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn/+merge/64294
Your team OpenERP Community is requested to review the proposed merge of lp:~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn into lp:openobject-addons/extra-trunk.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp
Reply | Threaded
Open this post in threaded view
|

Re: [Openerp-community] [Merge] lp:~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn into lp:openobject-addons/extra-trunk

Gustavo Adrian Marino
I wish to add something that is not technically wrong, but it could become a potential problem.

You are defining your user account.types on the account.xxxx namespace. I believe that could be a potential cause of problems on multi country databases (that is a multicompany installation with each company using a different template). If all localization templates uses the account.xxx namespace to define their own types, name crashing could become a real problem.

I suggest to keep account.types as locally defined items (that means using the module namespace) in order to prevent the a.m. danger
--
https://code.launchpad.net/~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn/+merge/64294
Your team OpenERP Community is requested to review the proposed merge of lp:~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn into lp:openobject-addons/extra-trunk.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp
Reply | Threaded
Open this post in threaded view
|

Re: [Openerp-community] [Merge] lp:~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn into lp:openobject-addons/extra-trunk

Gustavo Adrian Marino
In reply to this post by Numérigraphe
Review: Approve


--
https://code.launchpad.net/~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn/+merge/64294
Your team OpenERP Community is requested to review the proposed merge of lp:~openbmsjsc/openobject-addons/addons-extra-trunk-l10n_vn into lp:openobject-addons/extra-trunk.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-community
Post to     : [hidden email]
Unsubscribe : https://launchpad.net/~openerp-community
More help   : https://help.launchpad.net/ListHelp