tulip / ppe-logistics

Issue Tracking for PPELogistics.org
2 stars 1 forks source link

Only 2 Addresses per Customer #225

Open jackkrooss opened 3 years ago

jackkrooss commented 3 years ago

Changing things to allow only a Main and Delivery address to simplify things.

Changes include:

1: Customer Manager: add deliver address to the selected customer 2: Order manager : remove Update buttons and add save functionality to Open button NOTE: broke something with displaying display address for new orders

jackkrooss commented 3 years ago
  1. Remove the ability to update addresses from the Shipment Manager
  2. Add the shipping address to the Demand Manger. Note: I can not find the address variable here need help with this!
jackkrooss commented 3 years ago

So two problems:

A. When you create a new customer it does not populate the delivery address into the alt_address so when the order is first brought up there is no address

B. I would like to list the address on the Demand Manager (and I created a space to do so) but while I can get Region (which is deprecated) I can not get the address from the available variables.

jackkrooss commented 3 years ago

Jack Krooss 3:57 PM Anyway, I think that the place that whatever results in 'order detail 0 alt_address' on the Order Manager screen needs to get set in the Create/Edit screen is in the Submit Cart button trigger in the 'Transfer address to order' connector function. Wow. (edited)

Jack Krooss 4:15 PM OK, made a few small changes. In the Create/Edit Screen we are now calling them Main Address and Alternate Address. When the order is created it should save the Main address into whatever turns into 'order detail 0 alt_address' on the Order Manager screen. The dropdown should have both address options available. Once the address is selected and the order is reserved it is saved as the Delivery Address for that order and can not be changed. This si the address that will show up on the Demand Manager (not yet implemented) and the Shipment Manager as the "Address" and can not be changed.

I am going to need to look at the client ordering stuff to make sure this is consistent...aargh.

contextref commented 3 years ago

Whatever got accidentally pasted in this ticket was a "blowing off steam" conversation. I offer my apologies, and other than that all I can add is I am still around and intend to continue helping, and appreciate everyone's efforts in building and maintaining PPEL.

jackkrooss commented 3 years ago

My sincere apologies. This was my mistake. I intended to grab a couple paragraphs but accidentally posted an entire private DM thread, and quickly deleted it, but the email went out. As Ilya says I too am very appreciative of everyone's work. Again, very sorry.