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

Issue 32711002: trytond: Add tutorial documentation (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 years, 10 months ago by nicoe
Modified:
4 months ago
Reviewers:
acaubet, jonl, ced, perilla, udono, pokoli, rietveld-bot, reviewbot
Visibility:
Public.

Description

COLLABORATOR=jonl@advocatetax.com COLLABORATOR=sergi@koolpi.com COLLABORATOR=cedric.krier@b2ck.com

Patch Set 1 #

Total comments: 2

Patch Set 2 : Add first draft of displaying data page #

Patch Set 3 : Improve views topic #

Patch Set 4 : Add default values, on_change and functional fields #

Patch Set 5 : Add domains #

Total comments: 53

Patch Set 6 : Fix most comments #

Total comments: 1

Patch Set 7 : Update to tip, update code style and some minor improvements #

Total comments: 2

Patch Set 8 : Follow udo's suggestion #

Patch Set 9 : Use subfolder for tutorial, use cookiecutter and fix name and references (WIP) #

Patch Set 10 : Update more steps #

Total comments: 1

Patch Set 11 : Fix code #

Patch Set 12 : Add workflow and domain #

Patch Set 13 : Add states and extend #

Patch Set 14 : Update to tip #

Patch Set 15 : Add wizard #

Total comments: 10

Patch Set 16 : Fix remarks #

Total comments: 3

Patch Set 17 : Add report and table_query #

Total comments: 4

Patch Set 18 : Fix remarks #

Total comments: 34

Patch Set 19 : Fix remarks #

Patch Set 20 : Add link to topic #

Patch Set 21 : Fix table_query example #

Patch Set 22 : Update to tip #

Total comments: 24

Patch Set 23 : Fix comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1624 lines, -3 lines) Patch
M doc/index.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 3 chunks +6 lines, -1 line 0 comments Download
M doc/ref/models.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 1 chunk +1 line, -0 lines 0 comments Download
M doc/topics/modules/index.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 4 chunks +6 lines, -0 lines 0 comments Download
M doc/topics/reports/index.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 2 chunks +3 lines, -2 lines 0 comments Download
A doc/tutorial/index.rst View 1 2 3 4 5 6 7 8 1 chunk +10 lines, -0 lines 0 comments Download
A doc/tutorial/module/anatomy.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 1 chunk +23 lines, -0 lines 0 comments Download
A doc/tutorial/module/default_values.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 1 chunk +46 lines, -0 lines 0 comments Download
A doc/tutorial/module/domains.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 1 chunk +95 lines, -0 lines 0 comments Download
A doc/tutorial/module/extend.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 1 chunk +111 lines, -0 lines 0 comments Download
A doc/tutorial/module/function_fields.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 1 chunk +90 lines, -0 lines 0 comments Download
A doc/tutorial/module/index.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 1 chunk +40 lines, -0 lines 0 comments Download
A doc/tutorial/module/model.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 1 chunk +123 lines, -0 lines 0 comments Download
A doc/tutorial/module/on_change.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 1 chunk +66 lines, -0 lines 0 comments Download
A doc/tutorial/module/report.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 1 chunk +122 lines, -0 lines 0 comments Download
A doc/tutorial/module/setup.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 1 chunk +67 lines, -0 lines 0 comments Download
A doc/tutorial/module/setup_database.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 1 chunk +31 lines, -0 lines 0 comments Download
A doc/tutorial/module/states.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 1 chunk +92 lines, -0 lines 0 comments Download
A doc/tutorial/module/table_query.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 1 chunk +146 lines, -0 lines 0 comments Download
A doc/tutorial/module/view.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 1 chunk +212 lines, -0 lines 0 comments Download
A doc/tutorial/module/wizard.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 1 chunk +171 lines, -0 lines 0 comments Download
A doc/tutorial/module/workflow.rst View 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 1 chunk +163 lines, -0 lines 0 comments Download

Messages

Total messages: 75
nicoe
5 years, 10 months ago (2016-10-21 09:06:04 UTC) #1
pokoli
5 years, 10 months ago (2016-10-21 09:17:16 UTC) #2
reviewbot
patch is not applicable URL: https://codereview.tryton.org/32711002
5 years, 10 months ago (2016-10-21 09:22:49 UTC) #3
pokoli
https://tryton-rietveld.appspot.com/32711002/diff/1/doc/tutorial/creating_models.rst File doc/tutorial/creating_models.rst (right): https://tryton-rietveld.appspot.com/32711002/diff/1/doc/tutorial/creating_models.rst#newcode102 doc/tutorial/creating_models.rst:102: $ trytond-admin -d test --all For updating the module ...
5 years, 10 months ago (2016-10-21 10:11:46 UTC) #4
nicoe
https://tryton-rietveld.appspot.com/32711002/diff/1/doc/tutorial/creating_models.rst File doc/tutorial/creating_models.rst (right): https://tryton-rietveld.appspot.com/32711002/diff/1/doc/tutorial/creating_models.rst#newcode102 doc/tutorial/creating_models.rst:102: $ trytond-admin -d test --all On 2016/10/21 10:11:45, pokoli ...
5 years, 10 months ago (2016-10-21 15:30:07 UTC) #5
nicoe
Add first draft of displaying data page
5 years, 10 months ago (2016-10-21 15:32:35 UTC) #6
reviewbot
patch is not applicable URL: https://codereview.tryton.org/32711002
5 years, 10 months ago (2016-10-21 15:59:39 UTC) #7
pokoli
Improve views topic
5 years, 9 months ago (2016-11-04 14:04:23 UTC) #8
reviewbot
flake8 OK URL: https://codereview.tryton.org/32711002
5 years, 9 months ago (2016-11-04 14:31:46 UTC) #9
pokoli
Add default values, on_change and functional fields
5 years, 9 months ago (2016-11-05 08:30:19 UTC) #10
reviewbot
flake8 OK URL: https://codereview.tryton.org/32711002
5 years, 9 months ago (2016-11-05 08:54:16 UTC) #11
pokoli
Add domains
5 years, 6 months ago (2017-02-14 12:10:59 UTC) #12
reviewbot
flake8 OK URL: https://codereview.tryton.org/32711002
5 years, 6 months ago (2017-02-14 12:38:39 UTC) #13
perilla
https://tryton-rietveld.appspot.com/32711002/diff/80001/doc/tutorial/function_fields.rst File doc/tutorial/function_fields.rst (right): https://tryton-rietveld.appspot.com/32711002/diff/80001/doc/tutorial/function_fields.rst#newcode8 doc/tutorial/function_fields.rst:8: data in the database. For exemple, as we have ...
5 years, 6 months ago (2017-02-15 19:06:02 UTC) #14
ced
I find that the tutorial does not follow close enough the mq repository. But also ...
5 years, 6 months ago (2017-02-15 20:44:40 UTC) #15
pokoli
Fix most comments
5 years, 6 months ago (2017-02-16 08:51:42 UTC) #16
pokoli
https://tryton-rietveld.appspot.com/32711002/diff/80001/doc/tutorial/anatomy.rst File doc/tutorial/anatomy.rst (right): https://tryton-rietveld.appspot.com/32711002/diff/80001/doc/tutorial/anatomy.rst#newcode24 doc/tutorial/anatomy.rst:24: streamlined by the use of cookiecutter_. On 2017/02/15 20:44:39, ...
5 years, 6 months ago (2017-02-16 08:55:29 UTC) #17
reviewbot
flake8 OK URL: https://codereview.tryton.org/32711002
5 years, 6 months ago (2017-02-16 09:01:23 UTC) #18
ced
https://tryton-rietveld.appspot.com/32711002/diff/80001/doc/tutorial/anatomy.rst File doc/tutorial/anatomy.rst (right): https://tryton-rietveld.appspot.com/32711002/diff/80001/doc/tutorial/anatomy.rst#newcode24 doc/tutorial/anatomy.rst:24: streamlined by the use of cookiecutter_. On 2017/02/16 08:55:29, ...
5 years, 6 months ago (2017-02-16 10:01:08 UTC) #19
nicoe
On 2017/02/15 20:44:40, ced wrote: > I find that the tutorial does not follow close ...
5 years, 6 months ago (2017-02-16 10:17:47 UTC) #20
ced
On 2017/02/16 10:17:47, nicoe wrote: > On 2017/02/15 20:44:40, ced wrote: > > I find ...
5 years, 6 months ago (2017-02-16 10:45:41 UTC) #21
ced
On 2017/02/16 10:45:41, ced wrote: > > But if we do not use mq, who ...
5 years, 6 months ago (2017-02-16 10:48:06 UTC) #22
nicoe
* Cédric Krier [2017-02-16 11:45 +0100]: >On 2017/02/16 10:17:47, nicoe wrote: >>On 2017/02/15 20:44:40, ced ...
5 years, 6 months ago (2017-02-16 12:30:24 UTC) #23
ced
On 2017-02-16 13:30, Nicolas Évrard wrote: > > But if we do not use mq, ...
5 years, 6 months ago (2017-02-16 13:05:04 UTC) #24
nicoe
* Cédric Krier [2017-02-16 14:03 +0100]: >On 2017-02-16 13:30, Nicolas Évrard wrote: >> > But ...
5 years, 6 months ago (2017-02-17 17:12:25 UTC) #25
pokoli
5 years, 5 months ago (2017-02-20 09:11:55 UTC) #26
udono
https://codereview.tryton.org/32711002/diff/100001/doc/topics/views/index.rst File doc/topics/views/index.rst (right): https://codereview.tryton.org/32711002/diff/100001/doc/topics/views/index.rst#newcode32 doc/topics/views/index.rst:32: There is three types of views: There are the ...
2 years ago (2020-07-21 11:32:10 UTC) #27
pokoli
Update to tip, update code style and some minor improvements
11 months, 3 weeks ago (2021-08-26 07:56:00 UTC) #28
reviewbot
flake8 OK URL: https://codereview.tryton.org/32711002
11 months, 3 weeks ago (2021-08-26 08:06:57 UTC) #29
pokoli
On 2017/02/17 17:12:25, nicoe wrote: > * Cédric Krier [2017-02-16 14:03 +0100]: > >On 2017-02-16 ...
11 months, 3 weeks ago (2021-08-26 08:17:23 UTC) #30
udono
https://codereview.tryton.org/32711002/diff/352541002/doc/topics/views/index.rst File doc/topics/views/index.rst (right): https://codereview.tryton.org/32711002/diff/352541002/doc/topics/views/index.rst#newcode32 doc/topics/views/index.rst:32: There are different types of views: view types: https://codereview.tryton.org/32711002/diff/352541002/doc/topics/views/index.rst#newcode34 ...
11 months, 3 weeks ago (2021-08-26 08:26:53 UTC) #31
pokoli
Follow udo's suggestion
11 months, 3 weeks ago (2021-08-26 11:31:13 UTC) #32
reviewbot
flake8 OK URL: https://codereview.tryton.org/32711002
11 months, 3 weeks ago (2021-08-26 11:43:57 UTC) #33
ced
5 months ago (2022-03-16 17:25:00 UTC) #34
reviewbot
ERROR: /tmp/reviewbot-ftlh_w5k/doc/tutorial/module/domains.rst Imports are incorrectly sorted and/or formatted. URL: https://codereview.tryton.org/32711002
5 months ago (2022-03-16 17:29:48 UTC) #35
ced
5 months ago (2022-03-17 10:45:49 UTC) #36
reviewbot
ERROR: /tmp/reviewbot-642rwkga/doc/tutorial/module/domains.rst Imports are incorrectly sorted and/or formatted. URL: https://codereview.tryton.org/32711002
5 months ago (2022-03-17 11:05:49 UTC) #37
pokoli
https://codereview.tryton.org/32711002/diff/377031002/doc/tutorial/module/anatomy.rst File doc/tutorial/module/anatomy.rst (right): https://codereview.tryton.org/32711002/diff/377031002/doc/tutorial/module/anatomy.rst#newcode24 doc/tutorial/module/anatomy.rst:24: .. _`tryton template`: https://hg.tryton.org/cookiecutter this link is not used.
5 months ago (2022-03-17 14:07:13 UTC) #38
ced
5 months ago (2022-03-17 17:19:50 UTC) #39
reviewbot
ERROR: /tmp/reviewbot-m60ab_0v/doc/tutorial/module/domains.rst Imports are incorrectly sorted and/or formatted. URL: https://codereview.tryton.org/32711002
5 months ago (2022-03-17 17:34:52 UTC) #40
ced
4 months, 4 weeks ago (2022-03-20 22:20:40 UTC) #41
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 4 weeks ago (2022-03-20 22:36:22 UTC) #42
ced
4 months, 3 weeks ago (2022-03-23 22:36:45 UTC) #43
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 3 weeks ago (2022-03-23 23:04:13 UTC) #44
ced
4 months, 3 weeks ago (2022-03-27 14:16:48 UTC) #45
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 3 weeks ago (2022-03-27 14:32:52 UTC) #46
ced
4 months, 3 weeks ago (2022-03-27 21:10:15 UTC) #47
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 3 weeks ago (2022-03-27 21:39:43 UTC) #48
pokoli
https://codereview.tryton.org/32711002/diff/367001002/doc/tutorial/module/extend.rst File doc/tutorial/module/extend.rst (right): https://codereview.tryton.org/32711002/diff/367001002/doc/tutorial/module/extend.rst#newcode104 doc/tutorial/module/extend.rst:104: $ trytond does it make sense to include the ...
4 months, 3 weeks ago (2022-03-28 11:20:21 UTC) #49
ced
https://codereview.tryton.org/32711002/diff/367001002/doc/tutorial/module/extend.rst File doc/tutorial/module/extend.rst (right): https://codereview.tryton.org/32711002/diff/367001002/doc/tutorial/module/extend.rst#newcode104 doc/tutorial/module/extend.rst:104: $ trytond On 2022/03/28 11:20:21, pokoli wrote: > does ...
4 months, 3 weeks ago (2022-03-28 12:26:30 UTC) #50
ced
4 months, 3 weeks ago (2022-03-28 12:26:54 UTC) #51
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 3 weeks ago (2022-03-28 12:41:34 UTC) #52
ced
4 months, 2 weeks ago (2022-03-31 08:37:27 UTC) #53
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 2 weeks ago (2022-03-31 08:41:19 UTC) #54
acaubet
https://codereview.tryton.org/32711002/diff/403181002/doc/topics/modules/index.rst File doc/topics/modules/index.rst (right): https://codereview.tryton.org/32711002/diff/403181002/doc/topics/modules/index.rst#newcode86 doc/topics/modules/index.rst:86: >>>>>>> destination Missing merge https://codereview.tryton.org/32711002/diff/403181002/doc/tutorial/module/report.rst File doc/tutorial/module/report.rst (right): https://codereview.tryton.org/32711002/diff/403181002/doc/tutorial/module/report.rst#newcode44 ...
4 months, 2 weeks ago (2022-03-31 14:46:14 UTC) #55
ced
https://codereview.tryton.org/32711002/diff/403181002/doc/topics/modules/index.rst File doc/topics/modules/index.rst (right): https://codereview.tryton.org/32711002/diff/403181002/doc/topics/modules/index.rst#newcode86 doc/topics/modules/index.rst:86: >>>>>>> destination On 2022/03/31 14:46:14, acaubet wrote: > Missing ...
4 months, 2 weeks ago (2022-03-31 16:23:17 UTC) #56
ced
4 months, 2 weeks ago (2022-03-31 16:23:24 UTC) #57
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 2 weeks ago (2022-03-31 16:40:42 UTC) #58
udono
https://codereview.tryton.org/32711002/diff/380611002/doc/topics/modules/index.rst File doc/topics/modules/index.rst (right): https://codereview.tryton.org/32711002/diff/380611002/doc/topics/modules/index.rst#newcode86 doc/topics/modules/index.rst:86: >>>>>>> destination please clean https://codereview.tryton.org/32711002/diff/380611002/doc/tutorial/module/anatomy.rst File doc/tutorial/module/anatomy.rst (right): https://codereview.tryton.org/32711002/diff/380611002/doc/tutorial/module/anatomy.rst#newcode6 ...
4 months, 2 weeks ago (2022-03-31 21:29:00 UTC) #59
ced
https://codereview.tryton.org/32711002/diff/380611002/doc/tutorial/module/anatomy.rst File doc/tutorial/module/anatomy.rst (right): https://codereview.tryton.org/32711002/diff/380611002/doc/tutorial/module/anatomy.rst#newcode6 doc/tutorial/module/anatomy.rst:6: A Tryton module is a python module thus it ...
4 months, 2 weeks ago (2022-04-01 08:22:17 UTC) #60
ced
4 months, 2 weeks ago (2022-04-01 08:22:20 UTC) #61
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 2 weeks ago (2022-04-01 08:41:10 UTC) #62
ced
4 months, 2 weeks ago (2022-04-01 08:41:39 UTC) #63
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 2 weeks ago (2022-04-01 09:00:34 UTC) #64
ced
4 months, 2 weeks ago (2022-04-01 16:21:21 UTC) #65
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months, 2 weeks ago (2022-04-01 16:40:42 UTC) #66
ced
4 months ago (2022-04-12 09:00:32 UTC) #67
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months ago (2022-04-12 09:31:56 UTC) #68
pokoli
https://codereview.tryton.org/32711002/diff/401201002/doc/tutorial/module/default_values.rst File doc/tutorial/module/default_values.rst (right): https://codereview.tryton.org/32711002/diff/401201002/doc/tutorial/module/default_values.rst#newcode39 doc/tutorial/module/default_values.rst:39: python import, because the pool ensures that all of ...
4 months ago (2022-04-12 09:43:13 UTC) #69
ced
4 months ago (2022-04-12 10:26:26 UTC) #70
ced
https://codereview.tryton.org/32711002/diff/401201002/doc/tutorial/module/domains.rst File doc/tutorial/module/domains.rst (right): https://codereview.tryton.org/32711002/diff/401201002/doc/tutorial/module/domains.rst#newcode21 doc/tutorial/module/domains.rst:21: marked. A notification is displayed before saving. On 2022/04/12 ...
4 months ago (2022-04-12 10:26:28 UTC) #71
reviewbot
checks OK URL: https://codereview.tryton.org/32711002
4 months ago (2022-04-12 10:34:40 UTC) #72
pokoli
LGTM otherwise https://codereview.tryton.org/32711002/diff/401201002/doc/tutorial/module/workflow.rst File doc/tutorial/module/workflow.rst (right): https://codereview.tryton.org/32711002/diff/401201002/doc/tutorial/module/workflow.rst#newcode138 doc/tutorial/module/workflow.rst:138: </form> On 2022/04/12 10:26:28, ced wrote: > ...
4 months ago (2022-04-12 10:50:40 UTC) #73
rietveld-bot_tryton.org
New changeset 0cd242b9bfc4 by Cédric Krier in branch 'default': Add module tutorial https://hg.tryton.org/trytond/rev/0cd242b9bfc4
4 months ago (2022-04-12 11:09:23 UTC) #74
rietveld-bot_tryton.org
4 months ago (2022-04-12 11:09:34 UTC) #75
New changeset 6947e0b162af by Cédric Krier in branch 'default':
Add module tutorial
https://hg.tryton.org/tryton-env/rev/6947e0b162af
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld d9ca037-tainted