acteng / atip

Active Travel Infrastructure Platform
https://acteng.github.io/atip/
Apache License 2.0
22 stars 4 forks source link

Use new route-snappers, revamping how auto-named routes work #361

Closed dabreegster closed 1 year ago

dabreegster commented 1 year ago

This is a followup to #352. We're going to remove the route-info WASM library, which only has one useful working feature: auto-naming routes based on the two intersection endpoints. This PR changes that feature to instead use a new version of the route-snapper library. It also slightly improves the UX, by populating the name automatically when a route is first made. Users can still override the name, or later auto-fill it (to correct things after a split, for instance).

The benefits to dismantling the route-info service are:

On that last point, I'm going to remove the old route-info service next. It has to be separate, due to how tricky it is to upload the GH workflow script and handle all outstanding branches.

Robinlovelace commented 1 year ago

Awesome, great to see these quality of life improvements rolling in.