Update address and order validation

din-co / d2

Solidus-based commerce application

0 stars 2 forks source link

Closed cee-dub closed 8 years ago

cee-dub commented 8 years ago

Enforce deliverable shipping addresses, but be more lax about billing addresses. Fixes #155

beausmith commented 8 years ago

Is this ready for review? Took a gander. LGTM, but I'd love a walk though.

cee-dub commented 8 years ago

Yes. I would have shipped it if not sick.

Walk through: order, with a billing address outside our delivery area

beausmith commented 8 years ago

I was able to successfully test this on d2-staging-pr-239.

My order with billing address outside delivery zone…

cee-dub commented 8 years ago

Excellent. I say merge it and have someone else try on staging. If good, ship it. It's after noon.