Bugfix: Searching an address outside of an NC boundary silently fails

[ryanfchase]

Overview

We need to fix a regression so that users can still look up addresses outside of NC boundaries.

Action Items

• Modify behavior of the else block on L663 of Map.jsx
  • if ncIdOfAddressSearch is empty, still attempt to set the address state so that a geocoder pin is placed

Dev Resources

Useful Files and Components

• MapSearch.jsx (see props)
• Map.jsx (see L534, onGeocoderResult(result)

Mapbox Docs

• https://docs.mapbox.com/api/search/geocoding/

Resources/Instructions

• Bug Hunt report: Bug Hunt & Code Quality Live-Forever Ticket #1726 (comment)

Screen recording after proposed changes

[insert screenshot here]

[ryanfchase]

I'm opening this up as ready to work on. Smaller than 1pt, skipping dev lead review.

[bphan002]

ETA Oct Oct 17 Thursday

[bphan002]

@ryanfchase
I need a bit of clarification.

I was able to have the marker placed, but did we want it to zoom in on the location?
If we want that to still occur I'll have to look into it deeper since I believe the zoom is based on the neighborhood council.

[ryanfchase]

Adding this as a point of discussion for today's agenda. I'll add a response soon:

• 311 Data Weekly Meeting Agenda #1545 (comment)

[ryanfchase]

Design team to discuss in breakouts this evening. Follow up coming after meeting is over.

[ryanfchase]

@bphan002 for now, when an address is provided outside of an NC, can we fly to that address with a zoom level of 9? Design will be working on a ticket to decide how to message to the user that the search did not fall within an NC. Please move this ticket to "In Progress" if this clarification is sufficient.

Resources

• Example: Mapbox: FlyTo
• Map: https://docs.mapbox.com/mapbox-gl-js/api/map/#map#flyto
• https://docs.mapbox.com/mapbox-gl-js/api/properties/#cameraoptions
• https://docs.mapbox.com/mapbox-gl-js/api/properties/#animationoptions

[bphan002]

New ETA Friday Oct 25th EOD