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

Issue 32031002: country: Add proteus as required dependency

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 weeks ago by ced
Modified:
1 week, 3 days ago
Reviewers:
yangoon, reviewbot
Visibility:
Public.

Description

It is needed for the installed script trytond_import_zip issue6266

Patch Set 1 #

Total comments: 2

Patch Set 2 : No more need GeoName extra #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -4 lines) Patch
M INSTALL View 1 chunk +1 line, -0 lines 0 comments Download
M setup.py View 1 2 chunks +1 line, -4 lines 0 comments Download

Messages

Total messages: 8
ced
2 weeks ago (2017-02-13 14:46:30 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/32031002
2 weeks ago (2017-02-13 15:06:13 UTC) #2
yangoon
https://tryton-rietveld.appspot.com/32031002/diff/1/setup.py File setup.py (right): https://tryton-rietveld.appspot.com/32031002/diff/1/setup.py#newcode50 setup.py:50: requires = [get_require_version('proteus')] I think, that should be put ...
1 week, 4 days ago (2017-02-16 21:56:43 UTC) #3
ced
https://tryton-rietveld.appspot.com/32031002/diff/1/setup.py File setup.py (right): https://tryton-rietveld.appspot.com/32031002/diff/1/setup.py#newcode50 setup.py:50: requires = [get_require_version('proteus')] On 2017/02/16 21:56:43, yangoon wrote: > ...
1 week, 4 days ago (2017-02-16 22:23:50 UTC) #4
yangoon
On 2017/02/16 22:23:50, ced wrote: > https://tryton-rietveld.appspot.com/32031002/diff/1/setup.py > File setup.py (right): > > https://tryton-rietveld.appspot.com/32031002/diff/1/setup.py#newcode50 > ...
1 week, 3 days ago (2017-02-17 10:25:46 UTC) #5
ced
On 2017/02/17 10:25:46, yangoon wrote: > On 2017/02/16 22:23:50, ced wrote: > > > > ...
1 week, 3 days ago (2017-02-17 10:40:25 UTC) #6
ced
No more need GeoName extra
1 week, 3 days ago (2017-02-17 10:54:29 UTC) #7
reviewbot
1 week, 3 days ago (2017-02-17 11:05:58 UTC) #8
Sign in to reply to this message.

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