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

Issue 46691002: tryton-env: Add tab domain in url (Closed)

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

Description

issue7841 COLLABORATOR=cedric.krier@b2ck.com

Patch Set 1 #

Patch Set 2 : Remove unused variable #

Patch Set 3 : add changelog #

Patch Set 4 : Merge both reviews #

Total comments: 1

Patch Set 5 : Put changelog entry on top #

Total comments: 1

Patch Set 6 : Fix comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -1 line) Patch
M sao/CHANGELOG View 1 2 3 4 5 1 chunk +1 line, -0 lines 0 comments Download
M sao/src/sao.js View 1 2 3 4 5 1 chunk +1 line, -0 lines 0 comments Download
M sao/src/screen.js View 1 2 3 4 5 1 chunk +4 lines, -0 lines 0 comments Download
M tryton/CHANGELOG View 1 2 3 4 5 1 chunk +1 line, -0 lines 0 comments Download
M tryton/tryton/gui/main.py View 1 2 3 4 5 2 chunks +4 lines, -1 line 0 comments Download
M tryton/tryton/gui/window/view_form/screen/screen.py View 1 2 3 4 5 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 18
xcodinas
5 months ago (2018-11-13 16:43:55 UTC) #1
reviewbot
https://codereview.tryton.org/46691002/diff/1/tryton/gui/main.py#newcode899 tryton/gui/main.py:899: F841 local variable 'e' is assigned to but never used URL: https://codereview.tryton.org/46691002
5 months ago (2018-11-13 17:12:06 UTC) #2
xcodinas
Remove unused variable
5 months ago (2018-11-14 08:00:04 UTC) #3
reviewbot
flake8 OK URL: https://codereview.tryton.org/46691002
5 months ago (2018-11-14 08:13:48 UTC) #4
ced
Missing changelog
4 months, 4 weeks ago (2018-11-19 21:02:00 UTC) #5
xcodinas
add changelog
4 months, 4 weeks ago (2018-11-20 08:21:48 UTC) #6
reviewbot
flake8 OK URL: https://codereview.tryton.org/46691002
4 months, 4 weeks ago (2018-11-20 08:34:34 UTC) #7
xcodinas
4 months, 4 weeks ago (2018-11-20 14:50:35 UTC) #8
reviewbot
flake8 OK URL: https://codereview.tryton.org/46691002
4 months, 4 weeks ago (2018-11-20 15:07:00 UTC) #9
pokoli
https://codereview.tryton.org/46691002/diff/60001/sao/CHANGELOG File sao/CHANGELOG (right): https://codereview.tryton.org/46691002/diff/60001/sao/CHANGELOG#newcode3 sao/CHANGELOG:3: * Add tab domain in url should be on ...
4 months, 3 weeks ago (2018-11-22 11:03:34 UTC) #10
xcodinas
4 months, 3 weeks ago (2018-11-22 11:22:47 UTC) #11
reviewbot
flake8 OK URL: https://codereview.tryton.org/46691002
4 months, 3 weeks ago (2018-11-22 11:41:41 UTC) #12
ced
https://codereview.tryton.org/46691002/diff/80001/sao/src/screen.js File sao/src/screen.js (right): https://codereview.tryton.org/46691002/diff/80001/sao/src/screen.js#newcode1821 sao/src/screen.js:1821: dumps(this.attributes.tab_domain)]); I think it is more readable to break ...
4 months, 2 weeks ago (2018-12-04 12:53:50 UTC) #13
ced
2 months ago (2019-02-15 22:25:03 UTC) #14
reviewbot
flake8 OK URL: https://codereview.tryton.org/46691002
2 months ago (2019-02-15 22:41:43 UTC) #15
rietveld-bot_tryton.org
New changeset ea8b09617961 by Cédric Krier in branch 'default': Add tab domain in url https://hg.tryton.org/sao/rev/ea8b09617961
1 month, 3 weeks ago (2019-02-26 15:05:37 UTC) #16
rietveld-bot_tryton.org
New changeset 9db3d4f3eca5 by Cédric Krier in branch 'default': Add tab domain in url https://hg.tryton.org/tryton/rev/9db3d4f3eca5
1 month, 3 weeks ago (2019-02-26 15:05:41 UTC) #17
rietveld-bot_tryton.org
1 month, 3 weeks ago (2019-02-26 15:05:43 UTC) #18
New changeset 6ccbe3622b3c by Cédric Krier in branch 'default':
Add tab domain in url
https://hg.tryton.org/tryton-env/rev/6ccbe3622b3c
Sign in to reply to this message.

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