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

Tutorial - Error when click Save at Edit #725

Closed
CCH4GITHUB opened this issue Feb 13, 2017 · 17 comments
Closed

Tutorial - Error when click Save at Edit #725

CCH4GITHUB opened this issue Feb 13, 2017 · 17 comments

Comments

@CCH4GITHUB
Copy link

CCH4GITHUB commented Feb 13, 2017

Edit, Click Save gives rise to this error message

Error: 500 Internal Server Error
Error
at createConstructor (C:\Users\User\AppData\Roaming\npm\node_modules\geddy\lib\response\errors.js:68:34)
at new (C:\Users\User\AppData\Roaming\npm\node_modules\geddy\lib\response\errors.js:79:33)
at Object. (C:\Users\User\AppData\Roaming\npm\node_modules\geddy\lib\response\errors.js:21:10)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)

@CCH4GITHUB CCH4GITHUB changed the title Tutorial - Error when click Save at Eduit Tutorial - Error when click Save at Edit Feb 13, 2017
@ckhatton
Copy link
Member

Are you using node "~> 5.12.0"?

@CCH4GITHUB
Copy link
Author

CCH4GITHUB commented Feb 20, 2017 via email

@ckhatton
Copy link
Member

What's LTS? 😕

The node version cannot be any higher than v5 for Geddy to function. 🐙

@ckhatton
Copy link
Member

I get what you mean now, the node version has to be v5. You can use 'nvm' if you want to manage the versions.

@ckhatton
Copy link
Member

ckhatton commented Feb 22, 2017

@CCH4GITHUB Can I close this issue now? 🚪

@mde
Copy link
Contributor

mde commented Feb 25, 2017

@ckhatton, did we open a ticket for getting Ged working on the newest Nodes?

@ckhatton
Copy link
Member

I don't believe one has been created.

Can I possibly have a higher level of access? I would like to go through the issues and label them 'bug', 'feature', etc.

@mde
Copy link
Contributor

mde commented Feb 25, 2017

Done!

@ckhatton
Copy link
Member

Actually, doesn't look like it worked...

@mde
Copy link
Contributor

mde commented Feb 26, 2017

Look for an invitation to the group in e-mail.

@ckhatton
Copy link
Member

I still haven't received the email :( I have checked my spam folder as well.

@mde
Copy link
Contributor

mde commented Feb 28, 2017

Ugh. Can you look in your GitHub notifications to see if it's there? Maybe your notifications don't go to e-mail?

@mde
Copy link
Contributor

mde commented Mar 4, 2017

@ckhatton Is this the same error you saw trying to use Ged with the latest Node?

@mde
Copy link
Contributor

mde commented Mar 4, 2017

Looks like the breakage was caused by nodejs/node#6289, changing the querystring module to use null-proto objects. Fixed in mde/utilities@c906146 Update your deps, and things should work. Let me know if there's more breakage.

@mde mde closed this as completed Mar 4, 2017
@ckhatton
Copy link
Member

ckhatton commented Mar 6, 2017

Thinking about it, I think it was haha. An error would occur when saving a database record - So it does sound the same.

I kinda get the solution haha. Out of interest, how does Geddy link to the utilities repo? Do you have a asset call link to the raw source file in the utilities repo?

I have just tested the Geddy tutorial using Node v6, and it is working fine 😃👍

P.S:
I have noticed the page assets load in slower using Node v6. The background is white for a second, until the grey texture has loaded into the DOM. :suspect:

@mde
Copy link
Contributor

mde commented Mar 6, 2017

Geddy just uses the utilities lib as a standard CommonJS require. You can see all the stuff it uses in the package.json.

I haven't noticed any specific speed difference with Node 6, but in production, you probably wouldn't want to be serving the static assets with Node anyhow. You'd front your app with something like Ngnix to serve those.

@ckhatton
Copy link
Member

Geddy just uses the utilities lib as a standard CommonJS require.

Oh yeah, of course!

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

No branches or pull requests

3 participants