coopcycle / coopcycle-web

Logistics & marketplace platform. Only for worker-owned business.
https://coopcycle.org
Other
567 stars 129 forks source link

add "incident price change" column to orders export #4587

Open Paul-Eraman-CoopCycle opened 2 months ago

Paul-Eraman-CoopCycle commented 2 months ago

In orders export, the new price based on a reported incident is shown, but you cant see it was changed. propsal:

r0xsh commented 4 days ago

new column "incidents" : shows the name of the reported incident(s) attached to an order

Regarding the order-to-incident relationship mapping in the CSV export, we have two possible approaches:

  1. Create a separate CSV file specifically for incidents, maintaining the one-to-many relationship between orders and their associated incidents.

  2. Include all incidents within the order rows, where each order row would contain multiple incident entries (potentially making the row structure more complex).

Each approach has its trade-offs:

IMHO I'll go for the first approach since it's way easier to manipulate the data like so

Paul-Eraman-CoopCycle commented 4 days ago

Then let's go with multiple exports

However, for users could we have it be one button that exports one excel with two tabs?

Paul-Eraman-CoopCycle commented 4 days ago

Maybe the total incident price change and final price should be in the universal excel even if the list of incidents needs to be on a separate excel?

r0xsh commented 3 days ago

Yup, already added the incident price change + final price in the universal excel, the issue what with the incident titles

Paul-Eraman-CoopCycle commented 3 days ago

Beautiful :)