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

Issue 46731002: trytond: Allow to extend Field string and help (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 month, 3 weeks ago by ced
Modified:
5 days, 7 hours ago
Reviewers:
rietveld-bot, albert, reviewbot
Visibility:
Public.

Description

Those attributes are stored as StringPartitioned and each part creates its own translation record. The translated strings are constructed by concatenation of each translated part. issue8040

Patch Set 1 #

Patch Set 2 : flake8 #

Patch Set 3 : Update to tip #

Patch Set 4 : Fix translation type #

Unified diffs Side-by-side diffs Delta from patch set Stats (+131 lines, -73 lines) Patch
M CHANGELOG View 1 2 1 chunk +1 line, -0 lines 0 comments Download
M trytond/ir/model.py View 1 chunk +1 line, -1 line 0 comments Download
M trytond/ir/translation.py View 1 2 3 chunks +22 lines, -63 lines 0 comments Download
M trytond/model/fields/field.py View 1 2 3 3 chunks +28 lines, -8 lines 0 comments Download
M trytond/tests/test_tools.py View 1 2 chunks +54 lines, -1 line 0 comments Download
A trytond/tools/string_.py View 1 chunk +25 lines, -0 lines 0 comments Download

Messages

Total messages: 10
ced
1 month, 3 weeks ago (2019-01-24 18:11:32 UTC) #1
reviewbot
https://codereview.tryton.org/46731002/diff/1/trytond/tests/test_tools.py#newcode5 trytond/tests/test_tools.py:5: F401 'copy' imported but unused https://codereview.tryton.org/46731002/diff/1/trytond/tests/test_tools.py#newcode31 trytond/tests/test_tools.py:31: E501 line too long (90 > ...
1 month, 3 weeks ago (2019-01-24 18:38:33 UTC) #2
ced
flake8
1 month, 3 weeks ago (2019-01-24 20:28:18 UTC) #3
reviewbot
https://codereview.tryton.org/46731002/diff/20001/trytond/tests/test_tools.py#newcode30 trytond/tests/test_tools.py:30: E501 line too long (90 > 79 characters) https://codereview.tryton.org/46731002/diff/20001/trytond/model/model.py#newcode173 trytond/model/model.py:173: E501 line too ...
1 month, 3 weeks ago (2019-01-24 20:40:09 UTC) #4
albert
trytond/tools/string_.py was not properly uploaded
1 month, 3 weeks ago (2019-01-24 23:07:05 UTC) #5
ced
On 2019/01/24 23:07:05, albert wrote: > trytond/tools/string_.py was not properly uploaded I think rietveld does ...
1 month, 3 weeks ago (2019-01-25 08:27:02 UTC) #6
ced
Update to tip
1 week, 4 days ago (2019-03-09 14:15:07 UTC) #7
ced
Fix translation type
1 week, 4 days ago (2019-03-09 14:24:39 UTC) #8
reviewbot
https://codereview.tryton.org/46731002/diff/271081002/trytond/tests/test_tools.py#newcode30 trytond/tests/test_tools.py:30: E501 line too long (90 > 79 characters) https://codereview.tryton.org/46731002/diff/271081002/trytond/model/fields/field.py#newcode36 trytond/model/fields/field.py:36: E127 continuation line ...
1 week, 4 days ago (2019-03-09 14:48:08 UTC) #9
rietveld-bot_tryton.org
5 days, 7 hours ago (2019-03-15 12:02:43 UTC) #10
New changeset 11e271457e30 by Cédric Krier in branch 'default':
Allow to extend Field string and help
https://hg.tryton.org/trytond/rev/11e271457e30
Sign in to reply to this message.

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