1. Mailing Lists
  2. Human Resources
  3. Re: [OCA/hr] [10.0][FIX]hr_period (#527)

Human Resources mailing list archives

hr@odoo-community.org

Browse archives

Re: [OCA/hr] [IMP] hr_skill 11.0.2.0.0 (#503)

Re: [OCA/hr] [10.0][FIX]hr_period (#527)

Avatar

Re: [OCA/hr] [10.0][FIX]hr_period (#527)

by "Pedro M. Baeza" <notifications@github.com> - 11/12/2018 15:30:46

@pedrobaeza requested changes on this pull request.

This is not the best way to fix the tests, as this can lead to other side effects with demo data being manipulated by other modules through XML files with data. Please fix company creation instead. One possibility is to add post_install = True and at_install = False.

In hr_period/__manifest__.py:

> @@ -14,6 +14,7 @@
     'website': 'https://github.com/OCA/hr',
     'depends': [
         'hr_payroll',
+        'hr_payroll_account',

Why are you adding this dependency?

—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.