Skip to content

Conversation

@dwelch2344
Copy link

, instead of . - could happen to anyone ;)

dwelch2344 added 3 commits May 8, 2017 22:44
`,` instead of `.` - could happen to anyone ;)
@dwelch2344 dwelch2344 changed the title Bugfix in routes.js 404 Bugfix in routes.js 404; Support for reading request body May 9, 2017
@dwelch2344
Copy link
Author

I've been playing around with Restly and really like it. I saw the comment on #8 and figured I'd provide a solution I whipped together.

I'm basically adding a third (and thus optional) context param to routes that contains both the request (req) and response (res). I also had to add the express JSON bodyParser, but that should have little-to-no impact on the rest of the framework.

Happy to debate the impl and take a different approach if necessary. Thoughts?

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.

1 participant