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

[17.0] [MIG] mgmtsystem_audit #590

Merged
merged 107 commits into from
Oct 14, 2024
Merged

Conversation

m1k3lm
Copy link
Member

@m1k3lm m1k3lm commented May 21, 2024

Started the PR again to get rid of the mess at #561

Depends on:

  • mgmtsystem_nonconformity

@max3903
Copy link
Member

max3903 commented Oct 10, 2024

/ocabot migration mgmtsystem_audit

@max3903
Copy link
Member

max3903 commented Oct 10, 2024

@m1k3lm Can you rebase to get the dependency?

Maxime Chambreuil and others added 19 commits October 11, 2024 01:08
…he model and the views + #5119 - Grouped the list by procedure.
* [FIX] group_mgmtsystem_manager XML ID
* [IMP] mgmtsystem_kpi
* [FIX] Group references. Reported by michaelgiesbrecht on the forum.
* [ADD] Icons
* [IMP] Move audittrail dependency from mgmtsystem to requiring modules
… + Selections use words instead of letters; fixed AGPL-3 reference
…ded it.

mgmtsystem_nonconformity:

* o --> open c --> done state change.
* Migrating action relations to a many2many table.
* Concatening action comments in a single field

mgmtsystem_audit and mgmtsystem_review:

* o --> open c --> done state change.
In 7.0, it's necessary, in a view, to have the field present in a view in order to use it
in a domain. I added an invisible "state" field in relevant views to fix the errors.
* [FIX] Broken reference to wiki's group_id.

  In OpenERP 7, document_page had a parent_id instead of wiki's group_id. I've fixed the broken references that we had left in nonconformity and audit.

* [FIX] Fixed wrongful merge.

  The merge of nc-extend-7.0 was incorrect. It wrongly used the 6.1 version of mgmtsystem_audit.xml and mgmtsystem_nonconformity/mgmtsystem_nonconformity.xml. Fixed this.

* [CHG] Updated statusbar/buttons's layout to conform to OpenERP 7.0 style.

  Instead of a group at the bottom of the form, it's now a <header> tag at the top.

  Changed Actions, Audit, Nonconformity and Review.
* [CHG] Making audit trail rules an option. They will be automatically installed if the audittrail module is installed.
* [FIX] Bug 1137555
* [FIX] let the orm handle m2m fields; updating to new orm.Model notation.
…language + Use built-in "user" report variable instead of a custom one.
* module manifest and views to v7 format
* Add company_id and its default value
* Add company_id in views
* mgmtsystem_*_security.xml files
* Adapted audit to use openchatter
* Adding Audit Manager field
* Automatically add the Auditors, Auditees and Audit Manager to the follow list when creating Audit
* Audit closing posts to chatter; fix to autosub
* Added email templates that send one a week before and one 2 weeks before audit.
* Added object's direct link to emails.
oca-ci and others added 7 commits October 11, 2024 01:08
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: management-system-16.0/management-system-16.0-mgmtsystem_audit
Translate-URL: https://translation.odoo-community.org/projects/management-system-16-0/management-system-16-0-mgmtsystem_audit/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: management-system-16.0/management-system-16.0-mgmtsystem_audit
Translate-URL: https://translation.odoo-community.org/projects/management-system-16-0/management-system-16-0-mgmtsystem_audit/
Currently translated at 100.0% (111 of 111 strings)

Translation: management-system-16.0/management-system-16.0-mgmtsystem_audit
Translate-URL: https://translation.odoo-community.org/projects/management-system-16-0/management-system-16-0-mgmtsystem_audit/it/
Currently translated at 100.0% (111 of 111 strings)

Translation: management-system-16.0/management-system-16.0-mgmtsystem_audit
Translate-URL: https://translation.odoo-community.org/projects/management-system-16-0/management-system-16-0-mgmtsystem_audit/es/
@m1k3lm m1k3lm force-pushed the 17.0-mig-mgmtsystem_audit branch 2 times, most recently from defaeab to 565de4c Compare October 10, 2024 23:12
@m1k3lm
Copy link
Member Author

m1k3lm commented Oct 10, 2024

@max3903 Done!

@max3903
Copy link
Member

max3903 commented Oct 12, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 17.0-ocabot-merge-pr-590-by-max3903-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Oct 12, 2024
Signed-off-by max3903
@OCA-git-bot
Copy link
Contributor

@max3903 The merge process could not be finalized, because command oca-gen-addon-readme --if-source-changed --org-name OCA --repo-name management-system --branch 17.0 --addons-dir /tmp/tmptlo36l8w --commit failed with output:

Both .md and .rst found for DESCRIPTION. Please remove one of /tmp/tmptlo36l8w/mgmtsystem_audit/readme/DESCRIPTION.rst or /tmp/tmptlo36l8w/mgmtsystem_audit/readme/DESCRIPTION.md.

@m1k3lm
Copy link
Member Author

m1k3lm commented Oct 14, 2024

@max3903 removed .rst files.

@max3903
Copy link
Member

max3903 commented Oct 14, 2024

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 17.0-ocabot-merge-pr-590-by-max3903-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 964360f into OCA:17.0 Oct 14, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.