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

Issue 47351002: sao: Add views of the field in the loading strategy (Closed)

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

Description

This strategy limits the selection of other fields to load in bunch to those sharing at least one view. issue7359

Patch Set 1 #

Patch Set 2 : Update to tip #

Patch Set 3 : Update to tip #

Patch Set 4 : Update to tip #

Patch Set 5 : Add changelog #

Patch Set 6 : Fix missing variable declaration #

Unified diffs Side-by-side diffs Delta from patch set Stats (+63 lines, -34 lines) Patch
M CHANGELOG View 1 2 3 4 5 1 chunk +1 line, -0 lines 0 comments Download
M src/model.js View 1 2 3 4 5 2 chunks +30 lines, -25 lines 0 comments Download
M src/screen.js View 1 2 3 3 chunks +17 lines, -6 lines 0 comments Download
M src/view/form.js View 1 chunk +15 lines, -3 lines 0 comments Download

Messages

Total messages: 13
ced
5 months, 1 week ago (2018-04-16 15:55:44 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/47351002
5 months, 1 week ago (2018-04-16 15:57:41 UTC) #2
ced
Update to tip
4 months, 3 weeks ago (2018-04-30 13:56:38 UTC) #3
reviewbot
flake8 OK URL: https://codereview.tryton.org/47351002
4 months, 3 weeks ago (2018-04-30 13:58:48 UTC) #4
ced
Update to tip
4 months, 2 weeks ago (2018-05-08 11:36:45 UTC) #5
reviewbot
flake8 OK URL: https://codereview.tryton.org/47351002
4 months, 2 weeks ago (2018-05-08 12:13:33 UTC) #6
ced
Update to tip
3 months, 1 week ago (2018-06-11 21:28:19 UTC) #7
reviewbot
flake8 OK URL: https://codereview.tryton.org/47351002
3 months, 1 week ago (2018-06-11 21:32:32 UTC) #8
ced
Add changelog
3 months ago (2018-06-18 21:13:39 UTC) #9
reviewbot
flake8 OK URL: https://codereview.tryton.org/47351002
3 months ago (2018-06-18 21:34:34 UTC) #10
ced
Fix missing variable declaration
3 months ago (2018-06-19 13:21:40 UTC) #11
reviewbot
flake8 OK URL: https://codereview.tryton.org/47351002
3 months ago (2018-06-19 13:28:18 UTC) #12
rietveld-bot_tryton.org
3 months ago (2018-06-20 09:22:21 UTC) #13
New changeset 6a3d7715f9c7 by Cédric Krier in branch 'default':
Add views of the field in the loading strategy
https://hg.tryton.org/sao/rev/6a3d7715f9c7
Sign in to reply to this message.

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