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

Issue 46351002: www.tryton.org: Add release 4.8

Can't Edit
Can't Publish+Mail
Start Review
Created:
3 days, 9 hours ago by pokoli
Modified:
1 hour, 40 minutes ago
Reviewers:
timitos3000, ced, reviewbot
Visibility:
Public.

Patch Set 1 #

Patch Set 2 : Complete some items and add more screenshots #

Total comments: 15

Patch Set 3 : Fix timitos comments #

Patch Set 4 : Add introduction #

Total comments: 63
Unified diffs Side-by-side diffs Delta from patch set Stats (+553 lines, -0 lines) Patch
A files/images/sao_label_selected_records.png View Binary file 0 comments Download
A files/images/sao_locations_quantity_list.png View 1 Binary file 0 comments Download
A files/images/sao_sale_revenue_per_customer.png View 1 Binary file 0 comments Download
A files/images/sao_show_inactive_records.png View 1 Binary file 0 comments Download
A files/images/sao_sum_widget.png View Binary file 0 comments Download
A files/images/tryton_update_notification.jpg View 1 Binary file 0 comments Download
A posts/new-tryton-release-48.txt View 1 2 3 1 chunk +553 lines, -0 lines 63 comments Download

Messages

Total messages: 11
pokoli
3 days, 9 hours ago (2018-04-16 15:33:24 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/46351002
3 days, 8 hours ago (2018-04-16 15:57:22 UTC) #2
pokoli
Complete some items and add more screenshots
1 day, 12 hours ago (2018-04-18 12:05:49 UTC) #3
reviewbot
flake8 OK URL: https://codereview.tryton.org/46351002
1 day, 12 hours ago (2018-04-18 12:35:53 UTC) #4
ced
1 day, 11 hours ago (2018-04-18 13:38:10 UTC) #5
timitos3000
https://codereview.tryton.org/46351002/diff/20001/posts/new-tryton-release-48.txt File posts/new-tryton-release-48.txt (right): https://codereview.tryton.org/46351002/diff/20001/posts/new-tryton-release-48.txt#newcode24 posts/new-tryton-release-48.txt:24: * The clients shows next to the search input ...
14 hours, 8 minutes ago (2018-04-19 10:40:20 UTC) #6
pokoli
Fix timitos comments
10 hours, 19 minutes ago (2018-04-19 14:29:16 UTC) #7
reviewbot
flake8 OK URL: https://codereview.tryton.org/46351002
10 hours, 12 minutes ago (2018-04-19 14:36:15 UTC) #8
pokoli
Add introduction
10 hours, 7 minutes ago (2018-04-19 14:40:50 UTC) #9
reviewbot
flake8 OK URL: https://codereview.tryton.org/46351002
9 hours, 55 minutes ago (2018-04-19 14:53:18 UTC) #10
ced
1 hour, 40 minutes ago (2018-04-19 23:08:16 UTC) #11
https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
File posts/new-tryton-release-48.txt (right):

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:11: TUL, next versions will be only Python3
compatible.
TUL -> Tryton Unconference

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:13: This release is the first one to include
reporting capabilities. For now they
Indeed there was already some reporting before. So I would talk about a new way
of dynamic reporting.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:30: and to now that the model is deactivable.
to know

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:39: before opening the pop-up.
This is a more expected behavior for the user.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:41: * It will no more be possible to expand a
node that has too much records.
The news is on the present mode.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:44: be displayed in a list view.
which support loading records on the fly when displayed.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:52: removes also those data from the history
tables. But each module adds checks
But each modules -> And each modules

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:59: * The default search on party will now also
search on contact mechanism.
double "search"???
Why not:

The default search on party queries also the contact mechanisms.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:64: is possible now to define which email
address of a partyl should be used to
a party should

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:92: view.
We could have an image?

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:108: * The spell checking is now activated on
the browser, so fields with the
by the browser

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:116: if it is not valid. The edition is stopped
when the user click outside the
anywhere outside

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:125: * The Date and DateTime widghets the same
shortcuts available on tryton can now
widgets

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:125: * The Date and DateTime widghets the same
shortcuts available on tryton can now
The same shortcuts of the Date and DateTime widgets on the desktop client are
available on the web client.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:128: * Many2One fields now have a link on the
tree view which opens the related
are displayed on tree view as a clickable link which…

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:129: record form view to allow it's edition.
to allow quick edition.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:142: 
too much empty lines

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:167: period. The chart of accounts have been
updated to follow this design.
All chart

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:172: The new account_tax_cash module allows to
report taxes based on cash. A tax
`account_tax_cash`

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:173: group to report on cash basis are defined
on the *Fiscal Year* or *Period*.
The groups of taxes to report on cash…

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:191: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:196: invoice report will show the product
description instead of the line
the product name

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:199: * The reconciled date is now shown on the
invoice.
reconciliation date

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:201: * The Move lines now show which invoice
paid.
which invoice they pay.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:203: * An error will be raised when trying to
overpay an invoice.
is raised

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:208: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:217: closed, the payment is updated according if
the company win or lose.
won or losed

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:222: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:228: reconciliation wizard
<https://bugs.tryton.org/issue6917>`_ for the generated
on the generated

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:231: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:240: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:248: This release added on the
`products_by_location` method the possibility to
adds

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:267: * Until now, the assignation process was
not using children location but this
Must say it is for supplier return shipment

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:284: the product description to sale
description, as the product is now shown
product name

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:287: * In case a diferent product is shipped to
the customer if the invoice method
different

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:288: is based on shipment the shipped product
will be used on the invoice.
on shipment, this shipped

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:288: is based on shipment the shipped product
will be used on the invoice.
used for the invoice.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:289: Previously the sold product was always
used.
Previously only the initially sold product was used.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:291: * Now it is possible to edit the header
fields thanks a new Wizard which
thanks to the new wizard

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:312: :alt: Sale revenue per customer graph
It may be good to show also the sparklines

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:319: not shippping goods.
I do not understand.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:321: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:332: * Add product supplier on purchase line
The product supplier can be used now on the purchase line. This allow to display
the supplier definition of this product.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:338: the product description to pruchase
description, as the product is now shown
product name

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:347: warehouse than the warehouse of its
purchase request.
of the linked purchase requests

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:352: * This new module allows to manage requests
for quotation to different suppliers.
80 cols

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:356: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:360: * Now is possible to filter which type of
email should be used for sending
it is

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:370: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:372: ~~~~~~~~
empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:383: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:410: * The language formatting has been
simplified. Now it the language instance
Now the language

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:411: contains the `format`, `currency` and
`strftime` methods. A `get` method
contains -> exposes

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:435: * It is now possible for a module to only
if a specified sets of modules is
to register classes to the pool only if…

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:436: activated. This replace the silent skip
that existed previously. Existing
the previous silent skip.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:436: activated. This replace the silent skip
that existed previously. Existing
replaces

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:437: modules that were using this silent skip
must be updated to use the `depends`
relying on the old behavior must…

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:441: the XML record a field that is defined on
the optional module. We added a
fill from the XML record a value for a field

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:494: * Now is possible to extend with the same
it is

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:510: menu the button will be put.
will appear.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:542: * The invoice now has a method to add and
remove payments.
which should always be used.

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:544: 
too much empty line

https://codereview.tryton.org/46351002/diff/60001/posts/new-tryton-release-48...
posts/new-tryton-release-48.txt:553: 
trailing empty line
Sign in to reply to this message.

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