Open alank64 opened 11 years ago
Hello Andrew! Thanks for the pull request! I have a few comments:
Hey @therubymug @alank64
Can I help you with this PR? What there needs to do to merge it? It seems that this and next PRs have usefull staff and docs
Yes, very sorry, I made an attempt but got lost in the details of Travis and the likes. Now I'm overloaded with two tight deadlines on two simultaneous projects and won't have the time for a while.
You are welcome to go to my branch and clean/create and make the pull requests. Is that what you are asking? Or it might be easier to branch from Keymaker and cut/copy my pull requests into your branch, I think my branch is kinda messed up! Let me know, I'll add you as contributor to alank64/keymaker if you want.
On Tue, Apr 23, 2013 at 11:17 AM, Ivan Linko notifications@github.comwrote:
Hey @therubymug https://github.com/therubymug @alank64https://github.com/alank64
Can I help you with this PR? What there needs to do to merge it? It seems that this and next PRs have usefull staff and docs
— Reply to this email directly or view it on GitHubhttps://github.com/therubymug/keymaker/pull/8#issuecomment-16864696 .
On performing a cypher query, the results should contain the node_id or relationship_id in the results as the other non-cypher methods do. Without this there is no way to obtain the id in the results to perform further operations.
This pull req takes from the self parameter returned in the REST response and inserts the {node|relationship}_id into each of the results.