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

12.0 mgmtsystem nonconformity #247

Merged
merged 39 commits into from
Jul 25, 2019

Conversation

gurneyalex
Copy link
Member

@gurneyalex gurneyalex commented Jun 4, 2019

@oca-clabot
Copy link

Hey @gurneyalex, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/cla
Here is a list of the users:

Appreciation of efforts,
OCA CLAbot

@gurneyalex gurneyalex added this to the 12.0 milestone Jun 4, 2019
@OCA-git-bot OCA-git-bot mentioned this pull request Jun 4, 2019
15 tasks
@gurneyalex gurneyalex force-pushed the 12.0-mgmtsystem_nonconformity branch 3 times, most recently from 448fc69 to 958eff7 Compare June 5, 2019 07:09
@i-vyshnevska
Copy link
Member

@gurneyalex depends on not migrated document_page_procedure

@gurneyalex gurneyalex force-pushed the 12.0-mgmtsystem_nonconformity branch from 958eff7 to e04e36d Compare June 5, 2019 07:53
@i-vyshnevska
Copy link
Member

@gurneyalex document_page_procedure migrated #250
I will pick up your PR to finish migration

@i-vyshnevska i-vyshnevska force-pushed the 12.0-mgmtsystem_nonconformity branch from a0eed31 to 5e1d714 Compare June 5, 2019 12:02
@max3903
Copy link
Member

max3903 commented Jul 3, 2019

Still WIP?

@ernestotejeda ernestotejeda mentioned this pull request Jul 24, 2019
1 task
@ernestotejeda
Copy link
Member

I've done this PR (#256) that solves an error in mgmtsystem_action that causes this PR to fail. check it out, please. This PR is needed for #257

Maxime Chambreuil and others added 15 commits July 25, 2019 16:35
…ons.

[IMP] [#9439] Added the system reference field to the model and the views.
[IMP] efficiency -> effectiveness
[FIX] group_mgmtsystem_manager XML ID
[IMP] mgmtsystem_kpi
[FIX] Group references. Reported by michaelgiesbrecht on the forum.
[ADD] Icons
[IMP] Views
[ADD] NC : Workflow, actions_ids
[ADD] Action : Review fields
[REM] NC : Effectiveness fields
[IMP] Use workflow and improve filters.
[FIX] button_cancel invisible and system_id readonly on closed state.
[FIX] Can't group by procedure_ids
* ADD workflow reset option. ADD workflow transition from Open to Pending. CHG config tables are translatable. ADD pt translation.
* CHG process sequence logic. ADD approval records for each stage.
* CHG: finetunning user interface
* [ADD] Actions link back to NCs. [CHG] Actions UI improvements. [DEL] Action review not necessary. [ADD] NC on_change department fills automatically responsibles.
* ADD: communication & history tab for NCs
* ADD: keep history record for state changes and approvals

pt translations
…al module + qualification fields moved from feedback to NC; actions can be linked to projects
CHG: NC analysis is required at state 'p'

CHG: added parent to group NC Causes; removed NC Category

ADD: state 'analysis', between 'draft' and 'pending'

CHG: NC flow logic organized according to the typical sequence

CHG: small fixes and finetuning

CHG: marking Apps, security review, translation pt, finetuning
* licenses and dependencies
* Update hooks with addons modules from v7
* Permissions on files
* Add origins and causes data
* CHG Renamed 'feedback' to 'claim', see explanation  in __openerp__.py
* CHG pt_pt translation
* FIX Fixing fields read-only flags based on state
* CHG Improving action <-> NC relationship (wip)
* [FIX] fixes from review comments
* [FIX] missing context; instanciation of models
* [CHG] AGPL license; set verion to 1.0
* [CHG] Selections use words instead of letters; fixed AGPL-3 reference
* [FIX] .py files licenses to AGPL
* NameError: global name 'context' is not defined when resetting a NC
* Skeleton for migration script
* openupgrade analysis
[ADD] 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] I had accidentally broken the nonconformity/action migration shortly before commit.

[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.

[CHG] Adjusted mgmgsystem_nonconformity to OpenERP 7.0.

Like with mgmtsystem_action, the problem comes from changes made in OpenERP's messaging system.

[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.
"osv" was mistakenly removed from imports in nc-extend.
…with orm.except_orm()+ Fixed calls to action.case_open() which were made with explicit (cr, uid, id) args.

Now, these methods' arguments are implicitly set.
… replaced message history with chatter widget
Sandy Carter and others added 20 commits July 25, 2019 16:35
* Improvement to NC chatter state change messages.
* Buttons that change state backwards (towards draft) are no longer highlighted.
* Prefix overwriten for Nonconformity
* Fix to integrity check error
* revert unessessary_inherit to crm.claim
* [FIX] Fix PEP8 errors after running 'flake8 . --ignore E501,F401'
* [7.0.1.0 Migration scirpts] Added mgmtsystem_action. Made mgmtsystem_nonconformity system more robust.
* [FIX] Migration scripts no longer fully dependent on openupgrade
* [FIX] Bug #1267840 add values parametter to message_auto_subscribe
* Removed redundant import from openupgrade
Updated module as installable and removed depdencie on base_status

See

the base_status module is apparently gone so we have to work without it.
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-remove-base_status-tde/+merge/179887

Small fix to use stage_id.name instead of state

State doesn't exist anymore

Use stage_id for actions

Add support for settled, rejected instead of done/canceled... quite ugly

Separated python in each model file

Added missing methodcase_close_send_note for workflows

Updated model severity to V8

Port cause model to v8

Fix flake8 error

Fix small error in `cause` and upgraded `origin` to v8

Splitted xml views into 5 different xml files

Moved icon.png to static/description

Added tests and updated cause to v8

Added test origin and changed name of test class for cause

Removed severity redefined here

Added tests for create nonconformity

Ported fields to v8

Fix indent block

Removed ID and removed named lambdas

Simplify the name_get method

Get exceptions without importing orm

Removed size

Simplified name_get and removed id

Replace try except by self.assertRaise

Added group in forms

Reformat xml files removed tabs/space mix and 2 space indent

Fix tests

Added left and right to cause and origin

Add ondelete to parent_id

Fix tests

Improved name_get method and full coverage or origin/cause

Added workflow test

Added missing copyrights

Fix flake errors

Fix more flake errors

Some actions can only be done once actions, analysis

Fix even more flakes errors

Test reset button

Added missing fields (used in demo data)

Test state nonconformity name

Remove states as we are using stages on actions...

It would make more sense to add a property to stages that mark some
stages as ending stages instead of checking by names.

Use stage is ending and is starting instead of by name

Test analysis instead of evaluation

Fix flake8 errors

Added base security

Security rule in wrong project
* Move datas to their own folders
* Cleanup Nonconformity
* Use track and mail message subtypes instead of overriding functions
* Remove unused functions
* Cleanup and full use of v8 and track features
* Use a python context which does rollback
* Fix pep8
* Full test coverage of module
* Improving test coverage
* Fixing flake 8 error
* Fixing test error
* Remove unnecessary code
* adding drag and drop to kanban view
* Fixing flake 8 error
* Fixing tests errors
* add mgmtsystem_nonconformity_state access rule
* removing unused code
* Improving form and test
* Remove forced workflow logic
* Add kanban state
* Fix report state field
* Nonconformity view must load state data
* Adjust tests and make them pass
* Add kanban state (can be used for approvals)
* Fix duplicate stage/state tracking
* Fix report menu
* Add validation on close, requiring evaluation comment
* Replace age by days since last update
* Fix reports
* Reset Kanban State when changing Stages
* Prefer using the standard user_id field
* Properly implement the NC closing date
* Require action plan review for In Progress; Open Actions when In Progress
* Rename Pending Approval to Action Plan
…dd tests for Kanban state reset on stage change
MIG-mgmtsystem_nonconformity-to-V.-10.0
* [CHG] Moved immediate action field in description tab
* [ADD] 'name' attribute on description page to allow inheritance
* [FIX] Added specific tree view for causes with create=1 and delete=1 to allow a simple management user to select/deselect causes
* [FIX] Set create_date as readonly because it can't be changed anyway
Updated by Update PO files to match POT (msgmerge) hook in Weblate.
Currently translated at 95.0% (114 of 120 strings)

Translation: management-system-11.0/management-system-11.0-mgmtsystem_nonconformity
Translate-URL: https://translation.odoo-community.org/projects/management-system-11-0/management-system-11-0-mgmtsystem_nonconformity/it/
@pedrobaeza pedrobaeza force-pushed the 12.0-mgmtsystem_nonconformity branch from 5e1d714 to 215603e Compare July 25, 2019 14:36
@pedrobaeza
Copy link
Member

Rebased after making Travis green and rebased for reducing the huge useless commit history. There were already conflicts that wasn't correctly fixed in 11.0 migration, but I haven't touched that as it's too complicated to solve them now.

@pedrobaeza pedrobaeza force-pushed the 12.0-mgmtsystem_nonconformity branch from 215603e to 26a3ff3 Compare July 25, 2019 14:49
@pedrobaeza pedrobaeza force-pushed the 12.0-mgmtsystem_nonconformity branch from 26a3ff3 to da3d2e8 Compare July 25, 2019 15:28
@pedrobaeza pedrobaeza merged commit 40fd654 into OCA:12.0 Jul 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.