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

[18.0][MIG] mgmtsystem_review: Migration to 18.0 #612

Open
wants to merge 33 commits into
base: 18.0
Choose a base branch
from

Conversation

natuan9
Copy link
Contributor

@natuan9 natuan9 commented Nov 20, 2024

@max3903
Copy link
Member

max3903 commented Nov 30, 2024

/ocabot migration mgmtsystem_review

@OCA-git-bot OCA-git-bot mentioned this pull request Nov 30, 2024
15 tasks
@natuan9 natuan9 force-pushed the 18.0-mig-mgmtsystem_review branch from ff278dd to 3568c0a Compare December 4, 2024 07:24
Maxime Chambreuil and others added 21 commits December 4, 2024 14:25
* Initial commit. Import from extra-addons.
* cleanup
* Group references. Reported by michaelgiesbrecht on the forum.
* Icons
* Move audittrail dependency from mgmtsystem to requiring modules
…; fixed AGPL-3 reference + .py files licenses to AGPL
* [FIX] Bogus import in mgmtsystem_review.

  The wizard sub-module doesn't exist in this module and in openerp 7.0,
  it caused problems.

* [MIG] Added migration scripts for 0.1 --> 1.0 in the modules that needed 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.

* [FIX] Fixed "Unknown field state for domain" errors.

  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.

* [IMP] 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.
…installed if the audittrail module is installed.
* [FIX] Migration folder names
* label Poplicy n'est pas au bonne endroit
* [UPG] module manifest and views to v7 format
* [IMP] Add company_id and its default value
* [IMP] Add company_id in views
* [FIX] Missing object : review_line
* [IMP] mgmtsystem_*_security.xml files
* [FIX] PEP8 compliance after running flake8
Fix imports and replaced survey.responses by survey.user_input
Removed vim lines
[IMP] Add README.rst and data, views and models directories
Remove executable flag on python files and xml
Moved icon
[IMP] - mgmtsystem_review: Improved view and fixed typo.
[ADD] - mgmtsystem_review: Updated pot and added spanish translation.
[FIX] - mgmtsystem_review: Fixed translation template.
[REM] Migration files
[IMP] Description, version
[IMP] Add "Try me" button
[IMP] Remove osv
[FIX] Formatting in README.rst
[FIX] import models
[IMP] Migrate to new API
[FIX] AttributeError: 'module' object has no attribute 'Many2Many'
[FIX] Trailing comma after field definition
Currently translated at 100.0% (53 of 53 strings)

Translation: management-system-12.0/management-system-12.0-mgmtsystem_review
Translate-URL: https://translation.odoo-community.org/projects/management-system-12-0/management-system-12-0-mgmtsystem_review/pt_BR/
Currently translated at 100.0% (53 of 53 strings)

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

Translation: management-system-13.0/management-system-13.0-mgmtsystem_review
Translate-URL: https://translation.odoo-community.org/projects/management-system-13-0/management-system-13-0-mgmtsystem_review/
@natuan9 natuan9 force-pushed the 18.0-mig-mgmtsystem_review branch from 3568c0a to a434380 Compare December 4, 2024 07:25
odoo-addon-document_knowledge @ git+https://github.com/OCA/knowledge.git@refs/pull/503/head#subdirectory=document_knowledge
odoo-addon-document_page @ git+https://github.com/OCA/knowledge.git@refs/pull/504/head#subdirectory=document_page
odoo-addon-document_page_procedure @ git+https://github.com/OCA/management-system.git@refs/pull/610/head#subdirectory=document_page_procedure
odoo-addon-mgmtsystem_action @ git+https://github.com/OCA/management-system.git@refs/pull/605/head#subdirectory=mgmtsystem_action
odoo-addon-mgmtsystem_survey @ git+https://github.com/OCA/management-system.git@refs/pull/609/head#subdirectory=mgmtsystem_survey
odoo-addon-mgmtsystem_nonconformity @ git+https://github.com/OCA/management-system.git@refs/pull/611/head#subdirectory=mgmtsystem_nonconformity

Choose a reason for hiding this comment

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

surely all of them needed for this PR? I see only 2 links in PR description.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I missed adding some details to the PR description. I’ve now updated the test-requirements.txt and corrected the PR description.

@natuan9 natuan9 force-pushed the 18.0-mig-mgmtsystem_review branch from a434380 to 753f11d Compare December 12, 2024 05:02
Copy link
Member

Choose a reason for hiding this comment

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

@natuan9 To reset with the .md version.

@natuan9 natuan9 force-pushed the 18.0-mig-mgmtsystem_review branch from 753f11d to a384a63 Compare December 19, 2024 07:18
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.