[openstreetmap/openstreetmap-website] Fix roundabout exit text (#1808)

Jamie Guthrie notifications at github.com
Mon Apr 2 21:30:23 UTC 2018

This is basically a re-commit of #1805 , except I've removed the bit that separated out the `%{destinations}` template.

It now basically just fixes the ambiguous roundabout exit templating and refactors some of the code that builds the template name
i.e. https://www.openstreetmap.org/directions?engine=osrm_car&route=53.7338%2C-1.5258%3B53.6832%2C-1.4999#map=13/53.7086/-1.5171

(See instructions 7-16 - its not clear that every other instruction is an instruction to _leave_ the roundabout!)
You can view, comment on, or merge this pull request online at:


-- Commit Summary --

  * Refactored directions template building code
  * Fixed issue with some instruction template names having wrong variable order

-- File Changes --

    M app/assets/javascripts/index/directions/osrm.js (28)
    M config/locales/en.yml (91)

-- Patch Links --


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20180402/61e27543/attachment-0001.html>

More information about the rails-dev mailing list