Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Customer Returns Functionality #1306

Closed
metas-mk opened this issue Apr 11, 2017 · 8 comments
Closed

Customer Returns Functionality #1306

metas-mk opened this issue Apr 11, 2017 · 8 comments

Comments

@metas-mk
Copy link
Member

Is this a bug or feature request?

Feature Request

What is the current behavior?

Currently, we don't have an easy way to receive Customer Returns based on delivery Note lines and HU Assignments.

What is the expected or desired behavior?

Create the Functionality via Customer Returns Window or Material Receipt Window. Still in Discussion.

@metas-mk
Copy link
Member Author

Belongs to #547

@metas-rc metas-rc mentioned this issue May 16, 2017
metas-ts added a commit that referenced this issue May 16, 2017
[#1306](#1306) Customer Returns Functionality
@metas-dh metas-dh self-assigned this May 18, 2017
@metas-dh
Copy link
Member

Results from IT1
tested on mf15 Swing

  • HU creation for the customer return: TU as set in M_HU_PI_Item_Product, plus 1 LU (per product line): OK
  • movement for the HUs created: from Gebinde warehouse to warehouse selected in return: OK (had to create a warehouse for PIs first)
  • HU Assignment in the customer return, with the HUs created (LU): OK
  • invoice candidates created for the products customer return, with neg. qty (correct qty CU): OK
  • customer return for >1 product: >1 HU created, => 1 LU per product an resp. TU: OK
  • invoice candidates for the products customer return can be invoiced, are shown under "Kundenwarenrückgabe" with inout no., date, correct qty CU and price in invoice Jasper: OK

already reported problems (see task chat):

  • do not check the IsManualPackingMaterial checkbox, it is trying to create LU line which is not yet implemented
  • do not add lines with no M_HU_PI_Item_Product, the virtual PI part is not covered
  • qtyCU the user inserts is pretty much ignored in the HU creation, the LUTU config will be the expected one, based on the qtyEnteredTU and the Handling Unit Element Lager will have the max qty that fits into those TUs

new problems / questions:

  • customer return HU Assignment only shows the LU, no TU, OK??
  • TU created for return is shown with M_HU_PI_Item: "Virtual PI" in M_HU, only M_HU_Item shows the M_HU_PI_Item like e.g. G1: is that bc of the aggregation of HUs?
  • products to choose from for the inoutline in customer return: all available products can be selected, no matter if they were on customer pricelist or not: NOK imho
  • if the product chosen was not on the customer pricelist, then the IC created for this return has an error bc of that: NOK
  • pressing "Create Credit Memo" button in return creates an invoice, but it is a normal sales invoice, not a credit memo: will that button be used? if yes, this needs to be fixed

@metas-ts metas-ts reopened this Jun 9, 2017
@teosarca
Copy link
Member

teosarca commented Jun 9, 2017

rolled on w101

@metas-ts
Copy link
Member

CPed the latest into the release branch: https://jenkins.metasfresh.com/job/ops/job/cherry-pick-issue-script/167/

@metas-ts metas-ts reopened this Jun 14, 2017
@metas-lc
Copy link
Contributor

metas-lc commented Jun 14, 2017

Part 1: manual return
swing and webui:

  1. create customer return manually

  2. go to Kunden Rücklieferung and create a new one: Kundenwarenrückgabe

  3. add all mandatory fields

  4. add a new line, for a product e.g. P0001

  5. set uom, a package and a qty TU
    => qty is updated. OK

  6. change qty => qty TU is updated. Change qty TU => qty is updated OK

  7. complete
    => no distribution needed OK
    => got lines for TU and LU OK
    => 2 new HUs are created: LU and TU, both Active OK
    => movement for the HUs created: from warehouse set in my docu, to quality returns wh OK
    => HU Assignment in the customer return created OK (Followup created: Retracebility: Create comprehensible HU_Assignments for each document which involves an HU #1761)
    => jasper printed fine OK

  8. go to zoom in and select invoice candidates:
    => ICs for product + HUs OK

  9. create invoice
    => sales invoice candidate created, doctype credit memo, with neg. qty (correct qty CU), negative price, positive total price OK
    => qtys and hu qtys correctly set OK

  10. print jasper for your invoice
    => product is shown under "Kundenwarenrückgabe" with inout no., date, correct qty CU and price

  11. repeat the same testcase for multiple products (let's say 3 LUs, with 2, 10 and 1 TUs)
    => works the same OK
    => 1 invoice created with all 3 products OK

  • Part 2: return from shipment
    swing and webui
    on webui we can't do picking yet so i'll postpone the testing until it's possible
    swing:
  1. create a new sales order, add a product, complete
  2. go to picking terminal and pick the TUs
  3. go to shipment candidates and create inout with the picked qty. Do not complete the inout
    => inout created ok, button return from customer readonly
  4. complete inout
    => button return from customer editable
  5. select return from customer
    => hu editor opens (looks a bit weird OK? and the status is destroyed instead of shipped NOK)
  6. select all HUs and press on return from customer
    => nok error
    (after the fix from step 6 please also check how the lines are created)

@metas-lc
Copy link
Contributor

metas-lc commented Jul 4, 2017

IT

  • retested manually vendor return => works fine, as above OK
  • retested shipment (swing)
    select all HUs and press on return from customer => you get a notification that the vendor return is created
    open vendor return => doctype Kundenwarenrückgabe, product and TUs are set correctly.
    run jasper preview => works fine, TUs are set, qtys are ok
    => one issue: when you use split before/ after receipt you don't get all the HUs. Follow-up: Create link between a split HU and its original source model #1496

(also tested with different products/ TUs)

@metas-lc metas-lc closed this as completed Jul 4, 2017
metas-ts pushed a commit that referenced this issue May 7, 2020
shipment schedule editor: pimp layout + show packing instructions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants