search

bitExpert / pathfinder

[DEPRECATED] A PSR-7 based router
Apache License 2.0
5 stars 4 forks source link

"Optimize" the interaction with Adroit. #14

Closed shochdoerfer closed 8 years ago

shochdoerfer commented 8 years ago

See the discussion in the Adroit issue tracker. This will require a change in the Pathfinder repository as well.

dropdevcoding commented 8 years ago

I intentionally didn't implement more than target getter, params getter and a success flag, since I think there's no need to populate anything else outside routing. We could put in some more information though, but I don't think it should be necessary. What do you think?

Also I removed the tests for merging the route params with request params because this should be part of Adroit now (or any other implementation which is using Pathfinder).

shochdoerfer commented 8 years ago

Sounds good to me for the first version ;)