Closed niftylettuce closed 2 years ago
:+1:
I'll merge and publish this to NPM On Jun 19, 2014 8:58 AM, "Olivier Louvignes" notifications@github.com wrote:
[image: :+1:]
— Reply to this email directly or view it on GitHub https://github.com/visionmedia/express-resource/pull/90#issuecomment-46556572 .
:+1: gratz
:thumbsup:
:thumbsup:
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.
:+1: 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);
+1
@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.
+1 bro :yum:
Bump, I really want this inside my express 4 app :+1:
ping @niftylettuce
@fayimora @Anachron this isn't supported in 4.x because of the router issue I mentioned above
@niftylettuce it's okay, I don't use this library anymore, since I need 4.0 express. :+1:
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?
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.
+1 Just tested your branch with express 4.0 and it works fine for me