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

Support for Express 4.x #90

Closed
wants to merge 1 commit into from
Closed

Support for Express 4.x #90

wants to merge 1 commit into from

Conversation

niftylettuce
Copy link

No description provided.

@itayadler
Copy link

+1 Just tested your branch with express 4.0 and it works fine for me

@mgcrea
Copy link

mgcrea commented Jun 19, 2014

👍

@niftylettuce
Copy link
Author

I'll merge and publish this to NPM
On Jun 19, 2014 8:58 AM, "Olivier Louvignes" notifications@github.com
wrote:

[image: 👍]


Reply to this email directly or view it on GitHub
#90 (comment)
.

@vendethiel
Copy link

👍 gratz

@44203
Copy link

44203 commented Jun 24, 2014

👍

1 similar comment
@drewhamlett
Copy link

👍

@niftylettuce
Copy link
Author

One thing that isn't in yet is support for .resource on the new Router stuff in Express 4.x.

I think we should add that if we're going to keep using express-resource, otherwise you can't do a custom router and then do customRouter.resource as-is.

@bra1n
Copy link

bra1n commented Jul 15, 2014

👍
When will this be on NPM?

Also, the way I see it, the Readme should be updated, right?

var express = require('express')
  , Resource = require('express-resource')
  , app = express();

should be replaced with

var express = require('express')
  , app = express();
require('express-resource')(app);

@cjroth
Copy link

cjroth commented Jul 17, 2014

+1

@cjroth
Copy link

cjroth commented Jul 17, 2014

@niftylettuce this might not be an issue with your code but it looks like the "default" content type isn't working with Express 4 yet - in express 4, req.format is undefined which seems to be causing it to literally look for an "undefined" property instead of a "default" property. I will look into this further when I get a chance.

@sendyhalim
Copy link

+1 bro 😋

@Anachron
Copy link

Bump, I really want this inside my express 4 app 👍

@fayimora
Copy link

fayimora commented Jun 6, 2015

ping @niftylettuce

@niftylettuce
Copy link
Author

@fayimora @Anachron this isn't supported in 4.x because of the router issue I mentioned above

@Anachron
Copy link

Anachron commented Jun 6, 2015

@niftylettuce it's okay, I don't use this library anymore, since I need 4.0 express. 👍

@kevindoole
Copy link

I came here because i was curious to see if there was already a PR to deal with that express deprecated app.del notice. Otherwise, for my needs this package seems to be working nicely witrh express4.

In any case, is there any chance this package is gonna be updated in the future?

@niftylettuce
Copy link
Author

sorry if anyone wants to submit a new PR that is tested etc and works or something go for it then ping me and email niftylettuce@gmail.com - I've been using Koa a ton and haven't touched Express in forever.

This pull request was closed.
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.

None yet