webservice.js icon indicating copy to clipboard operation
webservice.js copied to clipboard

Please update createRouter.js for node 0.6

Open wolftankk opened this issue 13 years ago • 8 comments

Error: require.paths is removed. Use node_modules folders, or the NODE_PATH environment variable instead.

wolftankk avatar Nov 14 '11 09:11 wolftankk

same for me. It seems it imply updating journey and resourcer.

ghost avatar Dec 09 '11 14:12 ghost

It looks like journey has updated their code. Just resourcer now.

allenhartwig avatar Dec 13 '11 22:12 allenhartwig

I have a fork (https://github.com/dhendo/webservice.js) that is working against 0.6.5

I've made a number of changes, so Marak may or may not want to merge back into his.

Main changes:

  • Drop resourcer. Use revalidor instead (it was only using resourcer.validation anyway)
  • Updated the docs view. Added some jquery to allow expansion etc
  • Allow .pattern and .patterns to define a custom route pattern / pattens in a module method
  • Allow multiple captures in the patters - will be passed through in options._captures to the handling method.

YMMV

dhendo avatar Dec 15 '11 10:12 dhendo

@dhendo : thanks, nice work. Beware that "this.endpoint" is hardcoded in demoModule.js

ghost avatar Dec 15 '11 14:12 ghost

@dhendo - Nice! Would you be interested at all at trying to pull out Journey completely and replace with it https://github.com/flatiron/director/ ?

If not, I might be able to get some time in the next few weeks to do a new version.

Marak avatar Jan 16 '12 14:01 Marak

Yes, I may give that a shot soon.

I went part-way down that route initially, but then figured out what was going wrong with journey.

dhendo avatar Jan 16 '12 15:01 dhendo

Mostly got rewrite working with Director

https://github.com/Marak/webservice.js/commits/refactor

Marak avatar Mar 30 '12 01:03 Marak

Cool... I'll take a look

dhendo avatar Mar 30 '12 14:03 dhendo