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

Issue 50691002: trytond: Add timeout to Report.convert (Closed)

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

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M CHANGELOG View 1 chunk +1 line, -0 lines 0 comments Download
M trytond/report/report.py View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 5
ced
2 months, 1 week ago (2019-01-07 11:03:38 UTC) #1
reviewbot
https://codereview.tryton.org/50691002/diff/1/trytond/report/report.py#newcode20 trytond/report/report.py:20: E402 module level import not at top of file https://codereview.tryton.org/50691002/diff/1/trytond/report/report.py#newcode26 trytond/report/report.py:26: E402 module ...
2 months, 1 week ago (2019-01-07 11:12:11 UTC) #2
pokoli
LGTM
2 months, 1 week ago (2019-01-11 17:55:10 UTC) #3
rietveld-bot_tryton.org
New changeset 269b7c6621ea by Cédric Krier in branch 'default': Add timeout to Report.convert https://hg.tryton.org/trytond/rev/269b7c6621ea
2 months ago (2019-01-14 23:10:11 UTC) #4
rietveld-bot_tryton.org
2 months ago (2019-01-14 23:10:19 UTC) #5
New changeset ad0cac34bfca by Cédric Krier in branch 'default':
Add timeout to Report.convert
https://hg.tryton.org/tryton-env/rev/ad0cac34bfca
Sign in to reply to this message.

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