irc.libera.chat #tryton log beginning Wed Oct 4 12:10:01 AM CEST 2023 | ||
-!- tbruyere(~Thunderbi@mail.saluc.com) has joined #tryton | 06:18 | |
-!- mrichez(~Maxime@mail.saluc.com) has joined #tryton | 06:31 | |
-!- htgoebel(~htgoebel@p200300d5df316000d5b6b70db63ea593.dip0.t-ipconnect.de) has joined #tryton | 07:01 | |
-!- cedk(~ced@gentoo/developer/cedk) has joined #tryton | 07:02 | |
-!- ChanServ changed mode/#tryton -> +o cedk | 07:02 | |
-!- acaubet(~Thunderbi@194.224.31.235) has joined #tryton | 07:38 | |
-!- acaubet1(~Thunderbi@2a02:9130:fd00:bade:7c94:aaff:f0df:1df1) has joined #tryton | 08:03 | |
-!- udono(~udo@183-153-117-131.ip-addr.inexio.net) has joined #tryton | 08:06 | |
-!- acaubet(~Thunderbi@2a02:9130:fd00:bade:7c94:aaff:f0df:1df1) has joined #tryton | 08:10 | |
-!- acaubet(~Thunderbi@194.224.31.235) has joined #tryton | 08:43 | |
-!- acaubet(~Thunderbi@194.224.31.235) has joined #tryton | 08:54 | |
-!- cedk(~ced@gentoo/developer/cedk) has joined #tryton | 09:54 | |
-!- ChanServ changed mode/#tryton -> +o cedk | 09:54 | |
mrichez | hi, what could prevent to create a return shipment from a customer shipment (wizard to get data from selected shipment and create return shipment) to keep a track of shipment and return shipment on the same sale ? | 12:39 |
---|---|---|
pokoli | mrichez: nothing? | 12:40 |
mrichez | pokoli: :-) ? it seems strange to create a return sale (and it's difficult to understand what happens to a sale where there's a problem with a shipment) so why not a return shipment on the original sale | 12:43 |
pokoli | mrichez: normally the full sale is refunded to force the creation of a credit note, if you do not need a credit note you can just return directly the shipment | 12:44 |
pokoli | mrichez: Having a wizard to just return the shipment by creating a return shipment with the proper origins makes sense for me | 12:45 |
pokoli | If you set the proper origins then the sale workflow will recreate another customer shipment once the return is received | 12:45 |
mrichez | pokoli: ok... and also if invoice is not posted, will be invoice updated with the return shipment quantities ? | 12:46 |
pokoli | mrichez: if you have the sale_invoice_grouping module yes, otherwise an invoice will be created for each shipment | 12:47 |
mrichez | pokoli: ok.... i will think about it :-) we are using lots number and sale returns model needs to communicate lot number to select the same lot for the return shipment so we try to make something easier for the user.. | 12:49 |
pokoli | mrichez: but you must not set the lot number on creation but let the user input it when receiving the goods | 12:54 |
mrichez | pokoli: ok. we did the test with a customer return (we set sale_line as origin for the inventory move) and we notice that actual quantity on sale_line is the double instead of 0... bug ? (return move should be subtracted?) | 13:06 |
cedk | mrichez: you are not supposed to link a return move with a sale line | 13:08 |
cedk | also such move will be added to the quantity shipped | 13:09 |
mrichez | cedk: why? it allows to understand what happens to the sale... we ship some products, customer returns some for problem... | 13:09 |
mrichez | cedk: could we distinct customer move and return move with a negative quantity ? | 13:10 |
cedk | mrichez: origin is supposed to be filled by the system, it creates links for the system | 13:10 |
cedk | the system never creates a return move for a sale line with quantity > 0 | 13:11 |
mrichez | cedk: ok... maybe we could add some features on sale: (https://discuss.tryton.org/t/help-with-sale-procedure-and-sale-return/6563) indentify a sale return among the sales (in the treeview) , adding a one2many (like for amendments) to link sale returns | 13:13 |
pokoli | cedk: if you link both moves (out and return) to the same sale line it will just generate the missing quantity to fill the quantity requested by the customer | 13:25 |
cedk | pokoli: no | 13:31 |
pokoli | cedk: why not? | 13:32 |
cedk | pokoli: because the code does what it does, not what you think it does | 13:41 |
pokoli | cedk: but what is the rational to sum a retun move when when it is not really shipped? | 13:41 |
pokoli | cedk: yes, you are right about what the code does, but I can not understand why | 13:43 |
cedk | it is not possible to distinct all the moves | 13:47 |
pokoli | cedk: the form and to location is what distincs if its an customer out, or a return move, or any other move | 13:49 |
cedk | pokoli: you can not deal with all the cases | 13:50 |
pokoli | cedk: there are just three cases: Sum, Substract and ignore | 13:50 |
cedk | pokoli: you can not ignore | 13:50 |
pokoli | cedk: ok, then what we do with a production move related to a sale? | 13:51 |
cedk | that's the point | 13:51 |
pokoli | I do not understand why you said that it can not be ignored? | 13:52 |
cedk | pokoli: because user make the link | 13:53 |
pokoli | so? | 13:53 |
pokoli | cedk: event it can be the case that a third party module is creating the link on purpose | 13:54 |
pokoli | another point, is that if we want to prevent such mistakes we should consider making the origin readonly so the user will never update it, but that won't fix the case of third party modules | 14:01 |
mrichez | forget about use of origin: i'm using user admin for my tests... users don't have access to origin field... so this is a bad idea to use it... (or using it with a wizard) | 14:14 |
-!- cedk(~ced@gentoo/developer/cedk) has joined #tryton | 19:40 | |
-!- ChanServ changed mode/#tryton -> +o cedk | 19:40 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!