Skip to content

9.0 mgmtsystem nonconformity - #140

Merged
max3903 merged 51 commits into
OCA:9.0from
naousse:9.0-mgmtsystem_nonconformity
Jul 29, 2016
Merged

max3903 merged 51 commits into
OCA:9.0from
naousse:9.0-mgmtsystem_nonconformity

Conversation

@naousse

@naousse naousse commented Jun 9, 2016

Copy link
Copy Markdown
Contributor

No description provided.

@max3903 max3903 added this to the 9.0 milestone Jun 9, 2016
@coveralls

coveralls commented Jun 9, 2016

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling ddf5be1 on naousse:9.0-mgmtsystem_nonconformity into 18b9b4c on OCA:9.0.

@@ -218,27 +227,71 @@ def create(self, vals):

@api.multi
def message_auto_subscribe(self, updated_fields, values=None):

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@dreispt
Please Daniel, i need your help here. I'm unable to register a nonconformity because of this method that doesn't work.

@pedrobaeza pedrobaeza mentioned this pull request Jun 24, 2016
30 tasks
@@ -120,7 +138,7 @@ def _state_name(self):
track_visibility='onchange',
)
state_name = fields.Char(

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think this fieds is needed. State alone is enough.

@max3903

max3903 commented Jun 29, 2016

Copy link
Copy Markdown
Member

@naousse You can rebase this branch now.

@naousse
naousse force-pushed the 9.0-mgmtsystem_nonconformity branch from bbb5a1f to c03f57f Compare June 30, 2016 14:34
@max3903

max3903 commented Jul 1, 2016

Copy link
Copy Markdown
Member

@naousse

  • The create button has disappeared when you click on nonconformity in the menu
  • Drag and drop in the kanban view is not working
  • Merge [IMP] README.rst naousse/management-system#7 to improve README file
  • The "Close" button has disappeared. I had to click on "Closed" in the workflow steps to close the NC
  • Check the form view on branch 8.0, I think the action plan comments and effectiveness review fields have been improved

@max3903 max3903 self-assigned this Jul 1, 2016
@dreispt

dreispt commented Jul 27, 2016

Copy link
Copy Markdown
Member

@max3903 I didn't get this one:

When the NC is in progress state/stage, the actions of the plan should be opened.

@dreispt

dreispt commented Jul 27, 2016

Copy link
Copy Markdown
Member

@naousse naousse#11 implements it and a few other improvements/fixes - check the commit messages

@max3903

max3903 commented Jul 27, 2016

Copy link
Copy Markdown
Member
  • I was able to move the NC to the In Progress state/stage without the plan review. I should get an error message and be asked to enter it.
  • In the chatter of a nonconformity, the state/stage changes appears twice in each message
  • When the NC is in progress state/stage, the actions of the plan should be opened.
  • The menu item for report is not consistent with the others: Nonconformity --> Nonconformities

See http://3151194-140-e585e9.runbot2.odoo-community.org/web#id=2&view_type=form&model=mgmtsystem.nonconformity&menu_id=131&action=120

@dreispt

dreispt commented Jul 27, 2016

Copy link
Copy Markdown
Member

When the NC is in progress state/stage, the actions of the plan should be opened.

I don't understand this.

@max3903

max3903 commented Jul 27, 2016

Copy link
Copy Markdown
Member

@dreispt Actions are in draft until the action plan is reviewed. Once done, actions are opened and assignees can start working on them.

@dreispt

dreispt commented Jul 28, 2016

Copy link
Copy Markdown
Member

There you go: naousse#12 should fix those issues.
During tests, I figured that the "Pending Approval" stage wasn't intuitive, so I renamed it to "Action Plan".

@max3903

max3903 commented Jul 28, 2016

Copy link
Copy Markdown
Member
  •  I was able to close the NC with one action opened. I should get an error message and be asked to close every actions first (the immediate action and the ones of the action plan).

Otherwise 👍

@dreispt

dreispt commented Jul 28, 2016

Copy link
Copy Markdown
Member

There you go naousse#13

@dreispt

dreispt commented Jul 29, 2016

Copy link
Copy Markdown
Member

Coverage back to 100% 😺

@max3903

max3903 commented Jul 29, 2016

Copy link
Copy Markdown
Member

👍 Good work guys!

@max3903
max3903 merged commit 9780744 into OCA:9.0 Jul 29, 2016
@dreispt

dreispt commented Jul 29, 2016

Copy link
Copy Markdown
Member

🏆

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.

4 participants