nodejs_watchdog serves as a good example for people starting out with nodejs. Needs porting for D8.

CommentFileSizeAuthor
#3 interdiff-2277985-1-3.txt1.01 KBmrfelton
#3 nodejs-watchdog-2277985.3-d8.patch10.74 KBmrfelton
#2 interdiff-2277985-1-2.txt1.31 KBmrfelton
#2 nodejs-watchdog-2277985.2-d8.patch10.32 KBmrfelton
#1 nodejs-watchdog-2277985-d8.patch10.76 KBmrfelton

Comments

mrfelton’s picture

mrfelton commented
Status: Active » Needs review
StatusFileSize
new nodejs-watchdog-2277985-d8.patch10.76 KB

Initial stab at this. There are some TODOs and some FIXMEs. (will only work with all of the other D8 patches also applied)

mrfelton’s picture

mrfelton commented
StatusFileSize
new nodejs-watchdog-2277985.2-d8.patch10.32 KB
new interdiff-2277985-1-2.txt1.31 KB

Updated patch fixes issue with access check and message translation

mrfelton’s picture

mrfelton commented
StatusFileSize
new nodejs-watchdog-2277985.3-d8.patch10.74 KB
new interdiff-2277985-1-3.txt1.01 KB

Updated patch fixes issue with access check and message translation (last one had an extra change that should not have been there)

Anonymous’s picture

Anonymous (not verified) commented

looking good, thanks for working on this!

glekli’s picture

glekli commented
Status: Needs review » Fixed

I have made updates to the nodejs_watchdog module, and it is in a working condition now.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.