Merge lp://qastaging/~dreis-pt/project-service/7.0-fixcounters into lp://qastaging/~project-core-editors/project-service/trunk

Proposed by Daniel Reis
Status: Needs review
Proposed branch: lp://qastaging/~dreis-pt/project-service/7.0-fixcounters
Merge into: lp://qastaging/~project-core-editors/project-service/trunk
Diff against target: 128 lines (+107/-0)
4 files modified
.bzrignore (+1/-0)
project_issue_fixcount/__init__.py (+2/-0)
project_issue_fixcount/__openerp__.py (+35/-0)
project_issue_fixcount/project.py (+69/-0)
To merge this branch: bzr merge lp://qastaging/~dreis-pt/project-service/7.0-fixcounters
Reviewer Review Type Date Requested Status
Pedro Manuel Baeza Needs Information
Stefan Rijnhart (Opener) Pending
Review via email: mp+219558@code.qastaging.launchpad.net

Description of the change

In v7, Project kanban can take forever to load, due to inefficient implementation of the counters (Issues, Tasks and Documents).
AFAIK it has been fixed in v8, so this is mostly a backport for that work.

To post a comment you must log in.
41. By Daniel Reis

New project_issue_fixcounter module

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

Hi, Daniel, don't you think this fits best on OCB?

Regards.

review: Needs Information
Revision history for this message
Daniel Reis (dreis-pt) wrote :

I did hesitate on that. Let's ask for Stephan's opinion on it.

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

I suppose that he said that changes are too deep for OCB, and maybe he's right, but I don't think v7 is going to receive too much changes on this part to produce conflicts.

Let's see what he says.

Regards.

Revision history for this message
Daniel Reis (dreis-pt) wrote :

This project is now hosted on https://github.com/OCA/project-service. Please move your proposal there.
This guide may help you https://github.com/OCA/maintainers-tools/wiki/How-to-move-a-Merge-Proposal-to-GitHub

Unmerged revisions

41. By Daniel Reis

New project_issue_fixcounter module

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches