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

Issue 32031002: country: Add error message when running trytond_import_zip without proteus (Closed)

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

Description

Patch Set 1 #

Total comments: 2

Patch Set 2 : No more need GeoName extra #

Patch Set 3 : Add error message when running trytond_import_zip without proteus #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -4 lines) Patch
M scripts/trytond_import_zip View 1 2 1 chunk +10 lines, -4 lines 0 comments Download

Messages

Total messages: 11
ced
3 months, 1 week ago (2017-02-13 14:46:30 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/32031002
3 months, 1 week 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 ...
3 months, 1 week 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: > ...
3 months, 1 week 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 > ...
3 months, 1 week 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: > > > > ...
3 months, 1 week ago (2017-02-17 10:40:25 UTC) #6
ced
No more need GeoName extra
3 months, 1 week ago (2017-02-17 10:54:29 UTC) #7
reviewbot
https://codereview.tryton.org/32031002/diff/20001/setup.py#newcode83 setup.py:83: E501 line too long (85 > 79 characters) URL: https://codereview.tryton.org/32031002
3 months, 1 week ago (2017-02-17 11:05:58 UTC) #8
ced
Add error message when running trytond_import_zip without proteus
2 months, 3 weeks ago (2017-03-08 09:36:49 UTC) #9
reviewbot
flake8 OK URL: https://codereview.tryton.org/32031002
2 months, 3 weeks ago (2017-03-08 10:10:09 UTC) #10
rietveld-bot_tryton.org
2 months, 1 week ago (2017-03-21 22:35:06 UTC) #11
New changeset a4afb33b3e5c by C?dric Krier in branch 'default':
Add error message when running trytond_import_zip without proteus
http://hg.tryton.org/modules/country/rev/a4afb33b3e5c
Sign in to reply to this message.

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