-
Notifications
You must be signed in to change notification settings - Fork 93
Feature/v4 support #171
base: master
Are you sure you want to change the base?
Feature/v4 support #171
Conversation
Dear @ClaasBrueggemann, thank you very much for your Github contributions to GraphAware's Thank you very much in advance, Kind Regards, The GraphAware Team |
1 similar comment
Dear @ClaasBrueggemann, thank you very much for your Github contributions to GraphAware's Thank you very much in advance, Kind Regards, The GraphAware Team |
@ClaasBrueggemann Not sure it is really adding any support to Neo4j 4 ? |
@ikwattro It will not support multi databases and will still use the old api endpoints, but at least you can connect to the default database and run queries. Full support needs more work. |
@ClaasBrueggemann Hey, I am really interested in having multi-database support. Is there any way I can help to make it happen? I am even willing to write the PR if necessary, but it seems inevitable that this PR is merged first. |
@patrickkusebauch Hey, you can have a look at my forked repo: Especiallythe last commit(ClaasBrueggemann@beb5963) will be interesting for you. It adds The changes i made should be considered as a quick and dirty solution and can't be merged right away, because i copied the SessionInterface.php from another composer package to the main project and added a new method called useDatabase. Changing the SessionInterface should be done in the respective repository (https://github.com/graphaware/neo4j-php-commons). In this case somebody needs to update the bold connector, because it's using this interface too. (or use a different version tag) |
Add Neo4J 4.0 support using the HTTP connector.
Bold is not supported.