-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat(rest): Add Passport support #25
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
Comments
👍 |
Of course, it's valuable. |
Yes, this would be a good a addition to the generator. |
@nnance are you going to do this? Otherwise, I'm going to build this now, it would be quite valuable for me and since this is my favorite generator, I would try to make it as good as I can... (I'm quite new to the whole node world though...) |
If you want you can use this as a reference https://github.com/DaftMonk/angular-passport. I made it with an older version of the generator, but it might help you get started. |
@nnance I'm unable to find any repos from your side on github where I could contribute, but I'm willing to hack away and throw it out of the window if it won't fit in. Here are my thoughts:
is there interest in having a default "web side" for this? otherwise I would just "steal" from mean.io |
I've opened a pull request #36 that has the serverside basically done. Any feedback is appreciated. EDIT: I've made my way through the server side on this, client side needs feedback :) |
I was thinking of creating a pull request to add Passport as an OAuth module for server side rest application. Of course I would make it optional. Is this valuable?
The text was updated successfully, but these errors were encountered: