trailsjs / trailpack-hapi

:package: Hapi.js Trailpack
MIT License
19 stars 9 forks source link

warn user if an ORM is not installed, but footprints are enabled #33

Open tjwebb opened 8 years ago

tjwebb commented 8 years ago

If no ORM is installed but config.footprints.models is configured, then FootprintService will likely not exist and the footprint routes will crash.

jaumard commented 8 years ago

Maybe this can be done under trailpack-router instead of hapi/express/koa.... ?

jaumard commented 8 years ago

It can be done under validate method of trailpack-footprints

weyj4 commented 8 years ago

Does this mean we can close this issue w/r/t trailpack-hapi?

weyj4 commented 8 years ago

@jaumard can we close the issue?

jaumard commented 8 years ago

If we move the issue on another repo yes. There multiple choice for this :

trailpack-router can check this, warn user and ignore routes to not have crashing routes or trailpack-footprints can check this and warn user, but routes will crash