nxus / users

User management module for Nxus apps.
MIT License
0 stars 0 forks source link

Working version #3

Closed mjreich closed 8 years ago

loppear commented 8 years ago

Other than log comment, :+1:

loppear commented 8 years ago

So looking at the integration PR, I assume this purposely does not implement GET /forgot so that the including application can define that template... that's fine for now, but another case for figuring out how applications could selectively override templates and views routes, whether here or admin or .. would be nice if this could provide a default implementation etc, and the application could a) just provide an alternate template partial or b) fully replace the route with precedence over the default implementation.

And another case, so that the application could override the email template. With documentation of template names, presumably an application could app.get('templater').register('user-forgot-password', 'ejs', 'path/to/specific.ejs') so long as that register was after the normal gather of these. Might want to wrap that still, but that works for me for (a).

mjreich commented 8 years ago

@loppear yeah, the plumbing in there is ready for modules to define routes, with the expectation that there is a template using the 'default' namespace defined by the app. See _profileHandler in apiController.js.