Tryton - Issues

 

Issue7671

Title Possibly wrong account type for Belgian accounts
Priority bug Status resolved
Superseder Nosy List ced, dave, reviewbot, roundup-bot
Type behavior Components account_be
Assigned To ced Keywords review
Reviews 53421002
View: 53421002

Created on 2018-09-04.18:44:11 by dave, last changed by roundup-bot.

Messages
New changeset 51f70f20caac by Cédric Krier in branch 'default':
Fix type of account 32* and 33*
https://hg.tryton.org/tryton-env/rev/51f70f20caac
New changeset 2c90db2c2cd7 by Cédric Krier in branch 'default':
Fix type of account 32* and 33*
https://hg.tryton.org/modules/account_be/rev/2c90db2c2cd7
review53421002 updated at https://codereview.tryton.org/53421002/#ps1
msg44015 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2018-09-21.22:06:07
Indeed you are right, here is review53421002 that fix it.
msg43685 (view) Author: [hidden] (dave) Date: 2018-09-04.18:44:10
While looking at the account_be module I noticed that:

Accounts 32 (En-cours de fabrication), 320 and 329 have account type "...\A. Stocks\1. Approvisionnements".

Accounts 33 (Produits finis), 330 and 339 have account type "...\A. Stocks\2. En-cours de fabrication"

I am not sure whether this is incorrect, or not, as I am not familiar with Belgian accounts.  However I would have expected them to be "...\A. Stocks\2. En-cours de fabrication" and "...\A. Stocks\3. Produits finis" respectively.
History
Date User Action Args
2018-09-24 14:50:10roundup-botsetmessages: + msg44078
2018-09-24 14:50:00roundup-botsetstatus: testing -> resolved
nosy: + roundup-bot
messages: + msg44077
2018-09-21 22:31:21reviewbotsetnosy: + reviewbot
messages: + msg44018
2018-09-21 22:06:08cedsetstatus: unread -> testing
keyword: + review
nosy: + ced
messages: + msg44015
reviews: 53421002
assignedto: ced
2018-09-04 18:44:11davecreate

Showing 10 items. Show all history (warning: this could be VERY long)