Skip to content

[MIG] [13.0] mgmtsystem_hazard #293

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 72 commits into from
Mar 16, 2020
Merged

[MIG] [13.0] mgmtsystem_hazard #293

merged 72 commits into from
Mar 16, 2020

Conversation

flachica
Copy link
Contributor

@guadaltech @NachoAlesLopez Please, can you take a look to this?

Maxime Chambreuil and others added 30 commits March 10, 2020 17:29
[ADD] Initial commit. Import from extra-addons.
bde697b
[IMP] cleanup
f2416b8
[#6590] Fetch and evaluate the risk formula.
1e3e133
[#6590] Fetch the company_id from the res.users object.
ebd7812
[FIX] group_mgmtsystem_manager XML ID
7f5a0ad 
[IMP] mgmtsystem_kpi
[FIX] Group references. Reported by michaelgiesbrecht on the forum.
e357276
[ADD] Icons
6545bea
[IMP] Move audittrail dependency from mgmtsystem to requiring modules
547e651
[FIX[ licenses and dependencies
e41edc6
@dreispt
[CHG] AGPL license; set verion to 1.0
1cbd391
@dreispt
[FIX] .py files licenses to AGPL
3bfccb4
[FIX] Fix company_form_view's xpath expression so it works with OpenE…
a22d81d 
…RP 7's
@dreispt
[CHG] Making audit trail rules an option. They will be automatically …
1a606dc 
…installed if the audittrail module is installed.
[IMP] Upgrade to v7 and set default formula
c1b56f6
[IMP] Bump version and add translation file
99a0b12
[FIX] res.company form
c73c86a
[UPD] translation files
7877cb6
Launchpad automatic translations update.
a929de7
[IMP] Add company_id and its default value
514523c
[IMP] Add company_id in views
f650336
[ADD] mgmtsystem_*_security.xml files
da2bd49
[FIX] PEP8 compliance after running flake8
e373adb
Launchpad automatic translations update.
c243e38
[FIX] Syntax error
0992bb7
[FIX] Fix PEP8 errors after running 'flake8 . --ignore E501,F401'
f62f89d
Launchpad automatic translations update.
d6f3fac
[IMP] some demo data should not be recreated if we delete them
c6ca0bf
[FIX] fix security for some objects
ed1b470
Launchpad automatic translations update.
9394b99
@dreispt
Init 8.0 branch
dd8e46e
Olivier and others added 13 commits March 13, 2020 11:30
Add unittest to cover all code
66b1601
@oca-transbot
OCA Transbot updated translations from Transifex
319a16b
@sbidoul
[FIX] remove en.po that was erroneously created by transbot
da0b0ea
@oca-transbot
OCA Transbot updated translations from Transifex
65d59bd
@sbidoul
remove obsolete .pot files [ci skip]
196c756
@oca-travis
[UPD] Update mgmtsystem_hazard_risk.pot
fcf960e
@ngrandjean
[MIG] mgmtsystem_hazard_risk: Migration to 10.0
0b616d9
@oca-travis
[UPD] Update mgmtsystem_hazard_risk.pot
35a9572
@oca-transbot
Update translation files
b42bad5 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: management-system-10.0/management-system-10.0-mgmtsystem_hazard_risk
Translate-URL: https://translation.odoo-community.org/projects/management-system-10-0/management-system-10-0-mgmtsystem_hazard_risk/
@oca-travis
[UPD] Update mgmtsystem_hazard_risk.pot
2a7d1e5
@oca-transbot
Update translation files
691fcce 
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: management-system-10.0/management-system-10.0-mgmtsystem_hazard_risk
Translate-URL: https://translation.odoo-community.org/projects/management-system-10-0/management-system-10-0-mgmtsystem_hazard_risk/
[MIG] mgmtsystem_hazard_risk: Migration to 13.0
78e1eb2
Fix forgetting precommit
baaeb5e
@flachica flachica mentioned this pull request Mar 13, 2020
Closed
1 task
NachoAlesLopez
NachoAlesLopez reviewed Mar 14, 2020
View reviewed changes
Copy link

@NachoAlesLopez NachoAlesLopez left a comment

Choose a reason for hiding this comment

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

Functional Review: LGTM

@flachica
Copy link
Contributor Author

flachica commented Mar 15, 2020 •
edited
Loading

@NachoAlesLopez @max3903 Merged with hazard_risk maked but wait. I'll make it multi-company asap

NachoAlesLopez
NachoAlesLopez approved these changes Mar 15, 2020
View reviewed changes
Copy link

@NachoAlesLopez NachoAlesLopez left a comment

Choose a reason for hiding this comment

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

OK. Thank you for your work!

@flachica
Copy link
Contributor Author

@NachoAlesLopez @max3903 I have included the multi company. Is it necessary to warn of this?

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@max3903
Copy link
Member

max3903 commented Mar 16, 2020

/ocabot merge

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 13.0-ocabot-merge-pr-293-by-max3903-bump-no, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 003f49b into OCA:13.0 Mar 16, 2020
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 8508e28. Thanks a lot for contributing to OCA. ❤️

@flachica flachica deleted the 13.0-mig-mgmtsystem_hazard branch March 16, 2020 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@max3903 max3903 max3903 approved these changes

@dreispt dreispt dreispt approved these changes

@NachoAlesLopez NachoAlesLopez NachoAlesLopez approved these changes

Assignees

@flachica flachica

Labels
approved merged 🎉 migration needs review ready to merge
Projects
None yet
Milestone
13.0
Development

Successfully merging this pull request may close these issues.