-
Notifications
You must be signed in to change notification settings - Fork 72
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
fix: latitude and longitude mapping issue for version 5.0 #632
Conversation
Hello, thanks for your contribution! |
@fbiville I have sent the email with Neo4j CLA, if there is anything else that is required, please let me know, thanks. |
@yhhongyang 2 questions:
Thanks a lot! |
@fbiville For your 2 questions
|
@yhhongyang thanks but the emails still don't match |
@yhhongyang could you maybe send the CLA and amend the commit with your IBM email address? thanks! |
@fbiville I checked the email I sent earlier and found the problem and fixed it. Could you please check it again? Thanks |
@yhhongyang thanks! can you also please send the signed CLA with that email address please? |
I know you sent it but as I tried to say multiple times: the commit email and the email you use and mention for the CLA need to match. |
Did I send it right this time? |
Yes, thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @yhhongyang.
According to neo4j PointValue document, x represent longitude and y represent latitude
Change the mapping of latitude and longitude