Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Express4 route instead of app. Fixes ejs view #29

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

zevero
Copy link

@zevero zevero commented Sep 12, 2016

Integrating you gallery into my express app, res.render had problems finding my view directory.
After changing from app to router I am fine.
For standalone "views" and "view engine" are already set in examples/app.js so no need to set twice. I suspect you needed to set it again since a new app can not be aware of it. But router is.

@zevero
Copy link
Author

zevero commented Oct 24, 2016

Please merge ... there is no drawback :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants