refactor: Map new architecture PoC #4785
Conversation
| useEffect(() => { | ||
| if (region) updateRegion?.(region); | ||
| }, [region, updateRegion]); |
There was a problem hiding this comment.
After the tile server + vector source is in place, we can completely remove the updateRegion stuff :)
| openBottomSheet(() => { | ||
| return ( | ||
| <ScooterSheet | ||
| vehicleId={scooterFeature.properties.id} | ||
| onClose={() => { | ||
| setSharedState({selectedEntityId: undefined}); | ||
| closeBottomSheet(); | ||
| }} | ||
| onReportParkingViolation={() => {}} | ||
| /> | ||
| ); | ||
| }, false); |
There was a problem hiding this comment.
I think there is something very "un-react" about calling functions like this.
I would prefer dispatching an action that would set the app in the desired state 🤔
|
An interesting approach! |
PoC for this architecture:
