Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

support transaction options #494

Merged
merged 2 commits into from
Feb 19, 2018

Conversation

f5io
Copy link
Contributor

@f5io f5io commented Jan 4, 2018

Hey,

I tried to make this as backwards compatible as possible, no tests break, but I am aware this is because no transaction tests exist 😱.

I would be willing to take a crack at the tests if someone can give me some guidance as to what kind of assertions I should make, or whether it would be best to unit test and stub the endpoint?

@pluma
Copy link
Contributor

pluma commented Jan 11, 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.

@f5io
Copy link
Contributor Author

f5io commented Feb 19, 2018

CLA Signed :) sent on the 8th Feb

@pluma pluma merged commit 435d1ab into arangodb:master Feb 19, 2018
@pluma
Copy link
Contributor

pluma commented Feb 19, 2018

@f5io Thanks for submitting the CLA. Sorry this took a while, Cologne Carnival got in the way so I wasn't in the office last week.

I'll have to add some logic for backwards compatibility before releasing this.

pluma added a commit that referenced this pull request Feb 19, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants