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

Issue 26221002: carrier_weight: Add help text (Closed)

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

Description

Patch Set 1 #

Total comments: 4

Patch Set 2 : Add fields fix helps #

Total comments: 5

Patch Set 3 : Fix comments #

Total comments: 1

Patch Set 4 : Add missing unit in weight help #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -6 lines) Patch
M carrier.py View 1 2 3 4 chunks +12 lines, -6 lines 0 comments Download

Messages

Total messages: 13
xcodinas
1 year, 2 months ago (2017-03-06 12:58:06 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/26221002
1 year, 2 months ago (2017-03-06 13:08:11 UTC) #2
ced
https://tryton-rietveld.appspot.com/26221002/diff/1/carrier.py File carrier.py (right): https://tryton-rietveld.appspot.com/26221002/diff/1/carrier.py#newcode24 carrier.py:24: help="The uom that the carrier handles.") It is not ...
1 year, 1 month ago (2017-03-23 23:24:22 UTC) #3
xcodinas
Add fields fix helps
7 months, 3 weeks ago (2017-09-25 15:11:00 UTC) #4
reviewbot
flake8 OK URL: https://codereview.tryton.org/26221002
7 months, 3 weeks ago (2017-09-25 15:16:55 UTC) #5
ced
https://tryton-rietveld.appspot.com/26221002/diff/20001/carrier.py File carrier.py (right): https://tryton-rietveld.appspot.com/26221002/diff/20001/carrier.py#newcode24 carrier.py:24: help="The unit of mesure for the price list.") At ...
7 months, 3 weeks ago (2017-09-27 17:36:20 UTC) #6
xcodinas
Fix comments
7 months, 2 weeks ago (2017-10-04 11:08:09 UTC) #7
reviewbot
flake8 OK URL: https://codereview.tryton.org/26221002
7 months, 2 weeks ago (2017-10-04 11:19:36 UTC) #8
ced
https://codereview.tryton.org/26221002/diff/40001/carrier.py File carrier.py (right): https://codereview.tryton.org/26221002/diff/40001/carrier.py#newcode24 carrier.py:24: help="The weight criteria of the price list.") But now ...
6 months ago (2017-11-21 22:51:31 UTC) #9
xcodinas
Add missing unit in weight help
3 months ago (2018-02-13 11:58:51 UTC) #10
reviewbot
flake8 OK URL: https://codereview.tryton.org/26221002
3 months ago (2018-02-13 12:15:31 UTC) #11
ced
LGTM
2 months, 3 weeks ago (2018-02-25 22:33:01 UTC) #12
rietveld-bot_tryton.org
2 months, 3 weeks ago (2018-02-26 12:08:53 UTC) #13
New changeset 1308586eac66 by Xavier Codinas in branch 'default':
Add help text
http://hg.tryton.org/modules/carrier_weight/rev/1308586eac66
Sign in to reply to this message.

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