-
-
Notifications
You must be signed in to change notification settings - Fork 664
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
[MIG] Migration module hr_language in 10.0 #380
Conversation
acsonefho
commented
Oct 10, 2017
- Edit views
- Edit ir.model.access
- Split model files (one per model)
- Edit manifest
- Edit readme
- Edit translations
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not tested
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on runbot, and works fine.
Please consider reviewing OCA/project#328
"website": "http://www.savoirfairelinux.com", | ||
"depends": [ | ||
"hr", | ||
], | ||
'data': [ | ||
'security/ir.model.access.csv', | ||
'views/hr_language_view.xml', | ||
'views/hr_employee.xml', | ||
'views/hr_language.xml', | ||
], | ||
"demo": [], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
demo is empty and could be removed.
# -*- coding: utf-8 -*- | ||
# Copyright (C) 2017-Today: Odoo Community Association (OCA) | ||
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). | ||
from odoo import models, fields |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
imports should be alphabetically sorted
# -*- coding: utf-8 -*- | ||
# Copyright (C) 2017-Today: Odoo Community Association (OCA) | ||
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). | ||
from odoo import models, fields, tools |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
imports should be alphabetically sorted
@@ -250,6 +251,11 @@ msgid "Languages" | |||
msgstr "Langues" | |||
|
|||
#. module: hr_language | |||
#: help:hr.employee,language_ids:0 | |||
msgid "Languages of this employee" | |||
msgstr "Languages de cet employé" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Langues de cet employé".
"Language" translates to "Langue" in french, as "Langage" (and not "Language") has a different meaning.
So this should be fixed everythere in the french translation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixing the translation as I merge.
merged in 3fba7bd |
[BSSFL-441] Payment details on Quotation / Sale report