search

root-systems / cobuy

:seedling: :package: :stew: :family: Helping people buy good food at good prices, together.
GNU Affero General Public License v3.0
25 stars 4 forks source link

should be able to see order summary at any time #389

Open ahdinosaur opened 6 years ago

ahdinosaur commented 6 years ago

especially admins are keen to see what the order summary would look like if sent to suppliers

michael-smith-nz commented 6 years ago

Looks like the only change will need to use orderIntents instead of orderPlans as the data source. My understanding is that they are the same data shape

iainkirkpatrick commented 6 years ago

butting in here for a quick thought - i'm not sure my head is fully on this / understanding the purpose, but AFAIK it won't just be a case of switching out orderPlans for orderIntents - there is always only one OrderPlan created for each unique combination of an agent, product and order, whereas there may be many Intents from an agent per product per order. I.e. I can cast intents at all the price points on bananas in an order, but only one of those is the 'true' price point at any given time (depending on the intents of the rest of the group).

There is logic (read. getters) to figure out the 'true' price point on the client, used in the product list view to show the correct price and quantity... my bet would be to use those getters in this kind of pre-emptive OrderSummary view

sorry if this has already been covered and i'm just behind :sweat_smile:

iainkirkpatrick commented 6 years ago

@NotThatSmith you are off this ticket now right? can i take it over?

danalexilewis commented 6 years ago

@iainkirkparick yes Michael has shifted all hit work to the “tech between us” work for now. So take it over :)

iainkirkpatrick commented 6 years ago

@agentlewis in light of sprint planning this morning, is this still relevant?

danalexilewis commented 6 years ago

Yes but second milestone work.