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

[ADD] Migrated business_requirement_deliverable_categ module from v8 to v9 #29

Conversation

sudhir-serpentcs
Copy link
Contributor

WIP until #25 and #26 is merged

Migrated business_requirement_deliverable_categ module from v8 to v9.
@dreispt @pedrobaeza @jbeficent @moylop260 Please review.

Introduction
============

This module is part of a set of modules (`Business Requirements <https://github.com/OCA/business-requirement/blob/8.0/README.md>`_).
Copy link
Contributor

Choose a reason for hiding this comment

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

9.0

_inherit = "business.requirement.resource"

task_categ_id = fields.Many2one('project.tags',
string="Root Category for Tasks")
Copy link
Contributor

Choose a reason for hiding this comment

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

Category

@elicoidal
Copy link
Contributor

👍 (Waiting for #25)

@elicoidal elicoidal mentioned this pull request Sep 30, 2016
8 tasks
elicoidal pushed a commit that referenced this pull request Oct 4, 2016
…ble_cost

4 business requirement deliverable cost
astirpe and others added 4 commits October 4, 2016 14:00
Remove module project_categ from dependencies.
The OCA module project_categ is not yet ported to V9. Moreover the project.category model was removed from the standard V9, in favor of model project.tags.

With this PR I propose to not make use of project_categ at all.
Anyway still we should have an open discussion on how to handle the missing parent-child relation between project.tags.
[FIX] Remove project_categ from dependencies
[9.0] business_requirement_deliverable_categ: code review
@elicoidal
Copy link
Contributor

👍

@elicoidal
Copy link
Contributor

@sudhir-serpentcs can you add travis information to this repo?
Any reason why Runbot is failing?

Other than that, we could be merging this PR: @astirpe @pedrobaeza @dreispt @mmalorni

sudhir-serpentcs and others added 2 commits February 9, 2017 11:47
Added travis configuration file.
…o 9.0-migrate_business_requirement_deliverable_categ
@sudhir-serpentcs
Copy link
Contributor Author

Added travis and rebased the PR.
@dreispt @pedrobaeza @jbeficent @moylop260 @elicoidal

@elicoidal elicoidal merged commit c0774ba into OCA:9.0 Feb 9, 2017
@sudhir-serpentcs sudhir-serpentcs deleted the 9.0-migrate_business_requirement_deliverable_categ branch February 9, 2017 13:11
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.

None yet

4 participants