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

[IMP] Move closed flag of project task type direcly in project module instead of sale_service #8186

Closed

Conversation

adrienpeiffer
Copy link
Contributor

Currently, there is a close flag on task type model. This flag is used to define on a type of task if a task in this state is completed or not. But this flag is added by the sale_service module. However, it could be declared in the project module.

@sbidoul
Copy link
Contributor

sbidoul commented Sep 8, 2015

The closed flag on task type is of general interest in many modules. To get it today, we need to install sale_service which in turns pulls a lot dependencies which are not always desirable.

This PR moves the closed flag to the project module so it becomes more widely available.

Since project is already a dependency of sale_service this change is backward compatible.

👍

@pedrobaeza
Copy link
Collaborator

I also agree that this should be moved, but I'm not sure if stable policy allows to reallocate this field in v8. Anyway, for master and if possible for v8 👍

@sbidoul
Copy link
Contributor

sbidoul commented Sep 8, 2015

This does not change the database structure and the move would be virtually invisible to other addons so this should be ok wrt stable policy.

@rafaelbn
Copy link
Contributor

👍

@rafaelbn
Copy link
Contributor

@adrienpeiffer @sbidoul Odoo answer me. You can close this PR, Odoo will not merge it as:

Hello Rafael,

As stated in our contributing policy, we only accept proposals solving blocking issues in stable releases.

Improvements are not accepted in stable releases, such as 8.0.

For more information, please read:
https://github.com/odoo/odoo/wiki/Contributing#what-does-stable-mean

Do you think we can do this proposal to OCB? or independent module?

@sbidoul
Copy link
Contributor

sbidoul commented Dec 15, 2015

@rafaelbn I'm preparing a module project_stage_closed that I'll submit to OCA/project.

adrienpeiffer added a commit to acsone/timesheet that referenced this pull request Dec 16, 2015
adrienpeiffer added a commit to acsone/timesheet that referenced this pull request Dec 16, 2015
adrienpeiffer added a commit to acsone/timesheet that referenced this pull request May 20, 2016
@sbidoul
Copy link
Contributor

sbidoul commented May 27, 2016

project_stage_closed is merged in OCA/project. This PR is now unncessary. @adrienpeiffer can you close it?

@adrienpeiffer
Copy link
Contributor Author

Yes !

@sbidoul sbidoul deleted the 8.0-move-close-project-task-type-ape branch June 17, 2017 13:31
AaronHForgeFlow pushed a commit to ForgeFlow/hr-timesheet that referenced this pull request Mar 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants