Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(325)

Issue 45681002: account: Group by account and party when reconciling cancel moves (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 months, 1 week ago by pokoli
Modified:
3 months, 3 weeks ago
Reviewers:
rietveld-bot, ced, reviewbot
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 1

Patch Set 2 : Update existing move instead of creating a new one #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -6 lines) Patch
M move.py View 1 chunk +1 line, -1 line 0 comments Download
M tests/scenario_move_cancel.rst View 1 3 chunks +11 lines, -5 lines 0 comments Download

Messages

Total messages: 9
pokoli
4 months, 1 week ago (2018-08-08 13:50:35 UTC) #1
reviewbot
https://codereview.tryton.org/45681002/diff/1/move.py#newcode75 move.py:75: E131 continuation line unaligned for hanging indent https://codereview.tryton.org/45681002/diff/1/move.py#newcode421 move.py:421: E731 do not assign ...
4 months, 1 week ago (2018-08-08 14:00:27 UTC) #2
ced
https://codereview.tryton.org/45681002/diff/1/tests/scenario_move_cancel.rst File tests/scenario_move_cancel.rst (right): https://codereview.tryton.org/45681002/diff/1/tests/scenario_move_cancel.rst#newcode104 tests/scenario_move_cancel.rst:104: Create Move with two parties:: I would prefer to ...
4 months, 1 week ago (2018-08-08 14:18:41 UTC) #3
pokoli
Update existing move instead of creating a new one
4 months, 1 week ago (2018-08-08 15:59:14 UTC) #4
reviewbot
https://codereview.tryton.org/45681002/diff/20001/move.py#newcode75 move.py:75: E131 continuation line unaligned for hanging indent https://codereview.tryton.org/45681002/diff/20001/move.py#newcode421 move.py:421: E731 do not assign ...
4 months, 1 week ago (2018-08-08 16:10:02 UTC) #5
ced
LGTM, please commit the test separately for back-port.
4 months, 1 week ago (2018-08-08 19:46:03 UTC) #6
rietveld-bot_tryton.org
New changeset 0249bea0eef8 by Sergi Almacellas Abellana in branch 'default': Group by account and party ...
3 months, 3 weeks ago (2018-08-22 08:10:27 UTC) #7
rietveld-bot_tryton.org
New changeset 9aa3266f198a by Sergi Almacellas Abellana in branch 'default': Group by account and party ...
3 months, 3 weeks ago (2018-08-22 08:10:35 UTC) #8
rietveld-bot_tryton.org
3 months, 3 weeks ago (2018-08-26 19:13:41 UTC) #9
New changeset 8e27e0e3aeb4 by Sergi Almacellas Abellana in branch '4.8':
Group by account and party when reconciling cancel moves
https://hg.tryton.org/modules/account/rev/8e27e0e3aeb4


New changeset a6a425509add by Sergi Almacellas Abellana in branch '4.6':
Group by account and party when reconciling cancel moves
https://hg.tryton.org/modules/account/rev/a6a425509add


New changeset b8eb7d7a6742 by Sergi Almacellas Abellana in branch '4.4':
Group by account and party when reconciling cancel moves
https://hg.tryton.org/modules/account/rev/b8eb7d7a6742


New changeset eabbbfea6a60 by Sergi Almacellas Abellana in branch '4.2':
Group by account and party when reconciling cancel moves
https://hg.tryton.org/modules/account/rev/eabbbfea6a60


New changeset 3daf98a6d7a7 by Sergi Almacellas Abellana in branch '4.0':
Group by account and party when reconciling cancel moves
https://hg.tryton.org/modules/account/rev/3daf98a6d7a7
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 0147766