-
Notifications
You must be signed in to change notification settings - Fork 447
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
Support for Express 4.x? #462
Comments
+1 |
5 similar comments
+1 |
+1 |
+1 |
+1 |
+1 |
@bnoguchi, We are all very grateful for the work you have put into everyauth and we understand the difficult balance of work, life and a side project(s). Lots of developers have come to depend on this specific package and are feening to move their apps to Express 4. I understand time is probably the biggest factor but could I offer you $ to get it into your schedule? If you are already working on it, awesome, I still would contribute $ just because you deserve it. Any feedback would be super helpful. |
@CCVonLewin Thanks for the kick in the butt :) I started a new branch this past wknd. The new branch will support Express 4 and will also simplify some of the implementation of everyauth. I hope to finish it this weekend or by the subsequent weekend, time permitting. |
So I finished a first draft of a re-write of core everyauth today. See the example/ directory to get a feel for the changes. I only migrated the Facebook login module over. I need to migrate the other modules over and then clean up the re-write of core. If you'd like to help me migrate the other modules, send me a message, and I'll add you to the new https://github.com/everyauth group, where the modules will live. I'm going to hold off on publishing to npm for now, but try this branch out in the meantime. https://github.com/bnoguchi/everyauth/tree/0.5.x Here's a quick summary of what's different:
If you have any questions, let me know. It's late over here, but I wanted to send an update out to this thread about the changes and upcoming new release. |
+1 If anyone wants to try the 0.5.x preview, add dependency: "everyauth": "git://github.com/bnoguchi/everyauth.git#0.5.x" In case you do not know already how to add github dependencies, which I didn't :P |
Any progress on this? |
There's a branch that already works for express 4.x. I'll document and push Brian Noguchi On Fri, Nov 28, 2014 at 11:03 AM, Ilan Biala [email protected]
|
I assume it's the 1.x branch? Is documentation updated in that branch for the code re-write? Thanks in advance. |
@bnoguchi how about getting some help from a strong contributor? |
I'm doing something different from both See also #488 for my (original) plan. My branch is master...nponeccop:hotfix-0.4.10 If we end with something viable, we can release it to npm in a namespace such as |
Are there any plans to update this to support Express 4.x?
The text was updated successfully, but these errors were encountered: