Alex Wilson
Alex Wilson
Ok, I'll give it a go and add in a test case. This might take a bit because I'll need to install vagrant etc. As for ignoring records without id_property...
Just to be clear, when discussing "ignoring nodes without `id_property`" do you mean in the `association_proxy` method in particular? If so, I need to point out that not creating association...
@klobuczek Ok! I got the impression this change wasn't wanted. I'll split it up into 2 PRs, and force-push the smallest possible change to this branch.
Style changes extracted into #1547. Probably that one will have a little merge conflict after this change is merged, but I can fix that when the time comes.
Hi @klobuczek I will try to get it updated soon :) Thanks for getting back to me!
Another alternative here would be to change `replace_with` to always just fully delete before inserting the new relationships.
The relevant issue on neo4j-ruby-driver appears to be https://github.com/neo4jrb/neo4j-ruby-driver/issues/48
@klobuczek the build errored here due to [a failing wget command](https://travis-ci.com/github/neo4jrb/activegraph/jobs/508759012#L266) > The command "wget https://bootstrap.pypa.io/get-pip.py && python3 get-pip.py --user" failed and exited with 1 during . I'm not sure...
I appreciate the idea of working on making the API for the driver pleasant to use, but I think it's also important to keep it simple. This is the lowest...
One more plus for keeping the neo4j ruby driver very similar to other drivers is that it will be easier to port over security/bug fixes from those drivers as well.