Skip to content
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

Fixed wrong accessor in GraphEdgeCollectin.edge #501

Merged
merged 1 commit into from Mar 1, 2018

Conversation

Hunter21007
Copy link
Contributor

@Hunter21007 Hunter21007 commented Feb 2, 2018

the method returned undefined because result
of the edge operation does not returned graph.

Instead it returned edge.

changed from res.body.graph to res.body.edge
fixes: #499

the method returned undefined because result
of the edge operation does not returned graph.

Istead it returned edge.

changed from res.body.graph to res.body.edge
@Simran-B Simran-B requested a review from pluma February 2, 2018 09:57
@pluma
Copy link
Contributor

pluma commented Feb 2, 2018

Thank you for the PR. For legal reasons we need a signed Contributor License Agreement (CLA) before we can merge your code into the project.

You can download the form from our website: https://www.arangodb.com/documents/cla.pdf

Once you've filled out and signed it, please send it to us via e-mail: cla@arangodb.com or fax: +49-221-2722999-88.

@Hunter21007
Copy link
Contributor Author

Done :)

@Hunter21007
Copy link
Contributor Author

Via Fax

@pluma pluma merged commit 1fbd202 into arangodb:master Mar 1, 2018
@pluma
Copy link
Contributor

pluma commented Mar 1, 2018

Apologies for the wait. Your PR has been merged and will be shipped in the next release.

pluma added a commit that referenced this pull request Mar 1, 2018
@Hunter21007
Copy link
Contributor Author

Yeah, Thanks :)

@Hunter21007 Hunter21007 deleted the 499-graphedgecollection-edge branch March 2, 2018 22:20
@pluma
Copy link
Contributor

pluma commented Mar 6, 2018

FYI the changes have been released as 6.2.0.

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.

GraphEdgeCollection.edge returns undefined
2 participants