neo4j-contrib / graphgist-cms

http://graphgist.neo4j.com/
67 stars 24 forks source link

Gist submission page #6

Closed cheerfulstoic closed 9 years ago

cheerfulstoic commented 9 years ago

similar to this page

jexp commented 9 years ago

Please mark mandatory fields

and make it support pre-filled fields, like you would get at the "Submit" link from the footer of a graphgist page (e.g. from this one you get this submission link: https://docs.google.com/forms/d/1blgZoRZ6vLbpnqdJx3b5c4BkO_mgmD-hgdRQTMm7kc4/viewform?entry.718349727=http%3A%2F%2Fgist.neo4j.org%2F%3Fdropbox-14493611%252Fmeta_graph.adoc&entry.1981612324=Finally%3A%20The%20Meta-Graph&entry.1328778537=false&entry.507462214=false

cheerfulstoic commented 9 years ago

These bits are done. Looks like this:

/#/gists/submit?title=foo1&url=foo2&category=Education&submitter_name=foo3&submitter_twitter=foo4&description=foo5

Also, I was finally able to get Gist creation via the form to work (took a surprising amount of time). Next need to make a place where they can be approved

jexp commented 9 years ago

Cool ! What was the issue?

cheerfulstoic commented 9 years ago

There were multiple issues, actually ;)

I needed to add subpath.use(express.bodyParser()); to the api/app.js, but didn't realize for a while that I needed to add it to the subpath and not the app configure call. I also needed to modify and actually start using the Gists.create function and have it allow many params (though of course validating from the HTTP POST that only certain params are allowed).

Actually, looking at it it seems like it was just those too issues, but they took a while to figure out (and, of course, they seem so obvious now that I've found the solution...)

cheerfulstoic commented 9 years ago

Done with this now. It has a think you page with some basic prose and a link back to the site. I also changed the site to use the "status" property to determine if the gist should show on the site. The two states currently are 'live' and 'candidate'