Skip to content

resolve #485 #486

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

Merged
merged 1 commit into from
Sep 30, 2019
Merged

resolve #485 #486

merged 1 commit into from
Sep 30, 2019

Conversation

ryanmeador
Copy link
Contributor

Checking the type of the error. Without this check, the driver will try to divine meaning from Errors it didn't create, and the property might not be in a format it can understand.

…ck, the driver will try to divine meaning from Errors it didn't create, and the property might not be in a format it can understand.
@ryanmeador
Copy link
Contributor Author

I don't seem to have permission to access whatever went wrong with the build. If someone could tell me what's wrong, I'd be happy to fix it. I'd really like to see this bug fixed in the next 1.7.x release.

@ali-ince
Copy link
Contributor

Hi @ryanmeador,

Thanks for your contribution.

You should sign the CLA to get your changes to be tested by our CI and eventually get it merged. Please follow instructions mentioned here and then we'll be able to trigger the CI checks.

Thanks.

@ryanmeador
Copy link
Contributor Author

Hi @ali-ince. I submitted the CLA on August 31 at the same time I created the PR. If there's something wrong with it, please let me know.

@ryanmeador
Copy link
Contributor Author

Hi @ali-ince. Is there something I can do to help move this PR forward?

@ali-ince
Copy link
Contributor

It's all good @ryanmeador.

Thanks for your contribution!

@ali-ince ali-ince merged commit fd2da6e into neo4j:1.7 Sep 30, 2019
@ryanmeador
Copy link
Contributor Author

ryanmeador commented Sep 30, 2019 via email

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

Successfully merging this pull request may close these issues.

2 participants