PL regional partner search: search by ZIP only #25011
Conversation
breville
changed the title
| context: { | ||
| zip_code: zip_code | ||
| } | ||
| ) |
There was a problem hiding this comment.
Note - This means if the geocoder fails we could return nil (no result found) when really there is a result for that state. This is probably fine, and we'll be notified in honeybadger to look into it. I just want to call it out here that the user will see no regional partner found.
There was a problem hiding this comment.
Yes, I see Geocoder fails as the biggest risk to this change, but tracking with Honeybadger should help us understand if we need to change strategy. (We could show the state dropdown in such a case, but I wanted to avoid too many unusual code paths if it's not necessary.)
There was a problem hiding this comment.
FWIW we use it in many other places and at least for the ones I pay attention to like CSF workshop material orders, I have not seen it fail. AFAICT it's very reliable.
|
Tests added at #25047 |
The UI is updated to search by ZIP only. If a provider for that specific ZIP is not found, geocode the ZIP to get the state, and search for a provider covering that state instead.