search

18F / open-data-maker

make it easy to turn a lot of potentially large csv files into easily accessible open data
Other
200 stars 135 forks source link

Upgrade Padrino #311

Open yozlet opened 8 years ago

yozlet commented 8 years ago

Last time I tried to upgrade Padrino from 0.12.x to the latest version (0.13.2 at time of writing) it broke a couple of tests, and I didn't have time to investigate further.

yozlet commented 8 years ago

Further investigation with @kynetiv: there appears to be a weird change in how Padrino defines and calculates routes. This url_for call fails to find a matching route despite that very route being defined below. When we moved that route definition above the :endpoints route, the :endpoints route isn't found any more. So it may be a bug in either Padrino or Mustermann.