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

Issue 36281002: currency: Add help text

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

Description

Patch Set 1 #

Patch Set 2 : Fix some typos #

Total comments: 7

Patch Set 3 : Fix ced's comments #

Patch Set 4 : Add help for sign possition #

Total comments: 4

Patch Set 5 : Fix ced's comments #

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

Messages

Total messages: 12
pokoli
8 months ago (2017-03-27 11:27:26 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/36281002
8 months ago (2017-03-27 11:33:32 UTC) #2
pokoli
Fix some typos
8 months ago (2017-03-28 08:29:24 UTC) #3
reviewbot
flake8 OK URL: https://codereview.tryton.org/36281002
8 months ago (2017-03-28 08:38:17 UTC) #4
ced
https://codereview.tryton.org/36281002/diff/20001/currency.py File currency.py (right): https://codereview.tryton.org/36281002/diff/20001/currency.py#newcode40 currency.py:40: help="Add a separator for each number of digits in ...
1 month, 3 weeks ago (2017-10-02 18:47:31 UTC) #5
pokoli
Fix ced's comments
1 month, 3 weeks ago (2017-10-04 09:42:08 UTC) #6
reviewbot
flake8 OK URL: https://codereview.tryton.org/36281002
1 month, 3 weeks ago (2017-10-04 09:42:42 UTC) #7
pokoli
Add help for sign possition
2 weeks ago (2017-11-10 09:18:15 UTC) #8
reviewbot
flake8 OK URL: https://codereview.tryton.org/36281002
2 weeks ago (2017-11-10 09:36:01 UTC) #9
ced
https://codereview.tryton.org/36281002/diff/60001/currency.py File currency.py (right): https://codereview.tryton.org/36281002/diff/60001/currency.py#newcode26 currency.py:26: help="The numeric 3 chars ISO currency code.") it is ...
1 week, 4 days ago (2017-11-13 11:24:15 UTC) #10
pokoli
Fix ced's comments
1 week, 3 days ago (2017-11-14 11:59:27 UTC) #11
reviewbot
1 week, 3 days ago (2017-11-14 12:08:42 UTC) #12
Sign in to reply to this message.

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