🗺️ `Marketplace`: `DeliveryArea` #1136

zspencer · 2023-02-24T23:43:04Z

In the beginning, the Marketplace only supported per-Marketplace delivery pricing.

When chatting with Piikup, we learned that they prefer to price on a per-Marketplace basis based on which DeliveryArea the Shopper collects

Fauxsonas

See Marketplace Fauxsonas

Use Cases

🥔 Distributor adds DeliveryAreas
🥔✨ Marketplace: Removing DeliveryAreas #2014
🥔 Shopper sets DeliveryArea for Order
🥔Shopper pays DeliveryFee based upon DeliveryArea
🥔 Distributor sets OrderBy and DeliveryWindow for DeliveryArea
🥔Shopper sees DeliveryConstraints based upon DeliveryArea
🥔 Shopper may not see Marketplaces without a DeliveryArea

The text was updated successfully, but these errors were encountered:

- #1136 - #831 So, I totally didn't think about how delivery addresses are PII and probably should not be stored in plaintext! Womp. Womp. Womp. Now they ain't! We'll want to delete the `release:after_build` bits after a prod deploy.

- #1136 While collecting the address via Stripe was reasonable at the time, if we want to support dynamic pricing, we'll need the address ahead of time. This will also (theoretically) allow us to let people know when they are outside of the `Distributor` delivery area.

- #1136 - #831 So, I totally didn't think about how delivery addresses are PII and probably should not be stored in plaintext! Womp. Womp. Womp. Now they ain't! We'll want to delete the `release:after_build` bits after a prod deploy.

- #831 - #1136 Merge and deploy immediately after #1169

- #831 - #1136 Merge immediately after #1175

@KellyAH

- #1136 - #831 So, I totally didn't think about how delivery addresses are PII and probably should not be stored in plaintext! Womp. Womp. Womp. Now they ain't! We'll want to delete the `release:after_build` bits after a prod deploy. * `Marketplace`: Use `Lockbox.migrate` for safety @KellyAH found Lockbox's guide to migrating data: https://github.com/zinc-collective/convene/pull/1169/files#r1124067175 And it's way better!

* 🔐 `Marketplace`: Step 2 of `Lockbox` migration - #831 - #1136 Merge and deploy immediately after #1169

- #831 - #1136 Merge immediately after #1175

- #1136 While collecting the address via Stripe was reasonable at the time, if we want to support dynamic pricing, we'll need the address ahead of time. This will also (theoretically) allow us to let people know when they are outside of the `Distributor` delivery area.

#1173) - #1136 While collecting the address via Stripe was reasonable at the time, if we want to support dynamic pricing, we'll need the address ahead of time. This will also (theoretically) allow us to let people know when they are outside of the `Distributor` delivery area. * ✍️ `Marketplace`: Wordsmith! Co-authored-by: KellyAH <[email protected]> * ✨ `Marketplace`: Don't show `Cart` until there's a `delivery_address` * 🐞 `Marketplace`: Ensure `Guest`s can update `Cart` It's probably worth pushing some of these expectations down to the `cart_policy_spec` level, but the bug manifested at the controller tier so I wanted to catch it there. --------- Co-authored-by: KellyAH <[email protected]>

zspencer · 2023-03-19T18:05:34Z

From April:

We're going for Catering, so Restaurants and Food Makers and the pricing is based on City.
So if we are delivering from Cafe Gabriele or Understory those are Catering orders, and the prices are:

Oakland, Alameda, Berkeley and Emeryville is $50 for those cities

San Francisco, $75

- #1136 Co-authored-by: Zee Spencer <[email protected]> Co-authored-by: Ana <[email protected]> Co-authored-by: Dalton Pruitt <[email protected]>

- #1136 TODO: finish building create, edit, destroy flows. Co-authored-by: Zee Spencer <[email protected]> Co-authored-by: Ana <[email protected]> Co-authored-by: Dalton Pruitt <[email protected]>

* `Marketplace`: create `marketplace_delivery_areas` table. - #1136 Co-authored-by: Zee Spencer <[email protected]> Co-authored-by: Ana <[email protected]> Co-authored-by: Dalton Pruitt <[email protected]> * ✨`Marketplace` Started on the CRUD actions for `delivery_area`. - #1136 TODO: finish building create, edit, destroy flows. Co-authored-by: Zee Spencer <[email protected]> Co-authored-by: Ana <[email protected]> Co-authored-by: Dalton Pruitt <[email protected]> * - Display 'Marketplace::DeliveryArea' records on index view in a card component. - Fix `Marketplace::DeliveryAreasController` to create new DeliveryArea records. Co-authored-by: Zee Spencer <[email protected]> Co-authored-by: Dalton Pruitt <[email protected]> Co-authored-by: Ross Chapman <[email protected]> * add unit tests and delivery_area factory for #create and #new. Co-authored-by: Zee Spencer <[email protected]> Co-authored-by: Dalton Pruitt <[email protected]> Co-authored-by: Ross Chapman <[email protected]> * rubocop fix --------- Co-authored-by: Zee Spencer <[email protected]> Co-authored-by: Ana <[email protected]> Co-authored-by: Dalton Pruitt <[email protected]> Co-authored-by: Ross Chapman <[email protected]>

- #1325 - #1136 - #1185 Allowing a `Marketplace` to specify it's delivery fees, constraints, etc. and a `DeliveryArea` to override them was putting us in a bit of a mess from "how does data even?!" This consolidates everything onto `DeliveryArea`, yay!

- Prerequisite for #1420 - #1325 - #1136 - #1185 This does not get rid of the reading of the fields; but it does make it so they are no longer writable; which will make the "stop reading from them!" PR a bit tidier.

- #1325 - #1136 - #1185 Allowing a `Marketplace` to specify it's delivery fees, constraints, etc. and a `DeliveryArea` to override them was putting us in a bit of a mess from "how does data even?!" This consolidates everything onto `DeliveryArea`, yay!

- Prerequisite for #1420 - #1325 - #1136 - #1185 This does not get rid of the reading of the fields; but it does make it so they are no longer writable; which will make the "stop reading from them!" PR a bit tidier.

- #1325 - #1136 - #1185 Allowing a `Marketplace` to specify it's delivery fees, constraints, etc. and a `DeliveryArea` to override them was putting us in a bit of a mess from "how does data even?!" This consolidates everything onto `DeliveryArea`, yay!

zspencer · 2023-05-04T00:32:18Z

Re-opening because it does seem like we have more work to do to make this tight.

…1422) 🧹 `Marketplace`: `DeliveryConstraint`s only set in `DeliveryArea` - Prerequisite for #1420 - #1325 - #1136 - #1185 This does not get rid of the reading of the fields; but it does make it so they are no longer writable; which will make the "stop reading from them!" PR a bit tidier.

- #1325 - #1136 - #1185 Allowing a `Marketplace` to specify it's delivery fees, constraints, etc. and a `DeliveryArea` to override them was putting us in a bit of a mess from "how does data even?!" This consolidates everything onto `DeliveryArea`, yay!

`Marketplace`: `Delivery` constraints live on `DeliveryArea` - #1325 - #1136 - #1185 Allowing a `Marketplace` to specify it's delivery fees, constraints, etc. and a `DeliveryArea` to override them was putting us in a bit of a mess from "how does data even?!" This consolidates everything onto `DeliveryArea`, yay!

rosschapman · 2024-11-22T19:21:52Z

🧹 Relegated to cold storage.

zspencer mentioned this issue Feb 24, 2023

🥔🗜️✨ Gizmo: Marketplace #831

Closed

15 tasks

zspencer changed the title ~~🌱 Distributor sets ??? Distance ??? Delivery Fee?~~ Marketplace: 🌱 Distributor sets ??? Distance ??? Delivery Fee? Feb 24, 2023

zspencer mentioned this issue Feb 24, 2023

🔥👏🏾 Client: Piikup #832

Closed

1 task

zspencer changed the title ~~Marketplace: 🌱 Distributor sets ??? Distance ??? Delivery Fee?~~ Marketplace: Distance Based Delivery Pricing Feb 26, 2023

zspencer added the ✨ feature Reduces Client's Burden or Grants them Benefits label Feb 26, 2023

zspencer changed the title ~~Marketplace: Distance Based Delivery Pricing~~ Marketplace: Distance Based Delivery Pricing Feb 26, 2023

zspencer changed the title ~~Marketplace: Distance Based Delivery Pricing~~ 🌱 Marketplace: Distance Based Delivery Pricing Feb 26, 2023

zspencer added this to the 1.0 - Andromeda milestone Feb 26, 2023

zspencer mentioned this issue Mar 2, 2023

🔐 Marketplace: Encrypt Order#delivery_address! #1169

Merged

zspencer changed the title ~~🌱 Marketplace: Distance Based Delivery Pricing~~ 🌱 Marketplace: Dynamic Delivery Pricing Mar 2, 2023

zspencer mentioned this issue Mar 2, 2023

✨ Marketplace: Collect Shoppers Delivery Address before Checkout #1173

Merged

zspencer added a commit that referenced this issue Mar 3, 2023

🔐 Marketplace: Step 2 of Lockbox migration

68479ca

- #831 - #1136 Merge and deploy immediately after #1169

zspencer mentioned this issue Mar 3, 2023

🔐 Marketplace: Step 2 of Lockbox migration #1175

Merged

zspencer added a commit that referenced this issue Mar 3, 2023

🔐 Marketplace: Step 3 of encrypting delivery_address

4cf525f

- #831 - #1136 Merge immediately after #1175

zspencer mentioned this issue Mar 3, 2023

🔐 Marketplace: Step 3 of encrypting delivery_address #1176

Merged

zspencer added a commit that referenced this issue Mar 3, 2023

🔐 Marketplace: Step 2 of Lockbox migration (#1175)

92ba5ed

* 🔐 `Marketplace`: Step 2 of `Lockbox` migration - #831 - #1136 Merge and deploy immediately after #1169

zspencer added a commit that referenced this issue Mar 3, 2023

🔐 Marketplace: Step 3 of encrypting delivery_address

fbd0198

- #831 - #1136 Merge immediately after #1175

zspencer added a commit that referenced this issue Mar 3, 2023

🔐 Marketplace: Step 3 of encrypting delivery_address (#1176)

46e5898

- #831 - #1136 Merge immediately after #1175

KellyAH added a commit that referenced this issue Mar 23, 2023

Marketplace: create marketplace_delivery_areas table.

e8a68ce

- #1136 Co-authored-by: Zee Spencer <[email protected]> Co-authored-by: Ana <[email protected]> Co-authored-by: Dalton Pruitt <[email protected]>

KellyAH mentioned this issue Mar 28, 2023

✨🥔 Marketplace: Distributor creates DeliveryArea #1255

Merged

zspencer closed this as completed Apr 17, 2023

zspencer changed the title ~~✨ Marketplace: Delivery Area~~ 🥔✨ Marketplace: Delivery Area Apr 17, 2023

zspencer mentioned this issue Apr 30, 2023

🥗🧹 Marketplace: Delivery constraints live on DeliveryArea #1420

Merged

zspencer mentioned this issue Apr 30, 2023

🥗🥔🧹✨ Marketplace: DeliveryConstraints only set in DeliveryArea #1422

Merged

zspencer reopened this May 4, 2023

zspencer modified the milestones: 1.0 - Andromeda, 1.1 - B??? May 14, 2023

zspencer changed the title ~~🥔✨ Marketplace: Delivery Area~~ 🥔✨ Marketplace: DeliveryArea and DeliveryFee May 26, 2023

zspencer changed the title ~~🥔✨ Marketplace: DeliveryArea and DeliveryFee~~ 🥔✨ Marketplace: DeliveryArea May 26, 2023

zspencer mentioned this issue Aug 15, 2023

🌸✨ Marketplace: Shopper chooses DeliveryArea #1757

Merged

zspencer mentioned this issue Oct 1, 2023

✨ Marketplace: Show DeliveryFee when DeliveryArea is selected #1862

Merged

zspencer mentioned this issue Dec 10, 2023

🥔✨ Marketplace: Removing DeliveryAreas #2014

Open

7 tasks

zspencer mentioned this issue Feb 10, 2024

🐞 Marketplace: Buying Products - Can't Checkout when only one unarchived DeliveryArea #2198

Closed

zspencer changed the title ~~🥔✨ Marketplace: DeliveryArea~~ 🗺️ Marketplace: DeliveryArea Feb 17, 2024

zspencer mentioned this issue Feb 18, 2024

🥔✨ Marketplace: Buying 🥡 Products #1326

Closed

22 tasks

rosschapman closed this as completed Nov 22, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🗺️ `Marketplace`: `DeliveryArea` #1136

🗺️ `Marketplace`: `DeliveryArea` #1136

zspencer commented Feb 24, 2023 •

edited

Loading

zspencer commented Mar 19, 2023

zspencer commented May 4, 2023

rosschapman commented Nov 22, 2024

🗺️ Marketplace: DeliveryArea #1136

🗺️ Marketplace: DeliveryArea #1136

Comments

zspencer commented Feb 24, 2023 • edited Loading

Fauxsonas

Use Cases

zspencer commented Mar 19, 2023

zspencer commented May 4, 2023

rosschapman commented Nov 22, 2024

🗺️ `Marketplace`: `DeliveryArea` #1136

🗺️ `Marketplace`: `DeliveryArea` #1136

zspencer commented Feb 24, 2023 •

edited

Loading