Skip to content
This repository has been archived by the owner on Nov 7, 2018. It is now read-only.

Upgrade Padrino #311

Open
yozlet opened this issue Jun 20, 2016 · 1 comment
Open

Upgrade Padrino #311

yozlet opened this issue Jun 20, 2016 · 1 comment

Comments

@yozlet
Copy link
Contributor

yozlet commented Jun 20, 2016

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
Copy link
Contributor Author

yozlet commented Jun 23, 2016

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.

@kynetiv kynetiv mentioned this issue Dec 28, 2016
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@yozlet