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

Support sort with deleteOne, docs -> document/documents #88

Merged
merged 6 commits into from
May 2, 2023

Conversation

vkarpov15
Copy link
Collaborator

What this PR does:

Various fixes for new jsonapi changes, taking into account stargate/data-api#399, stargate/data-api#141 + stargate/data-api#402.

Also sundry other improvements:

  1. _.keys() -> Object.keys(). No need to use _.keys(), Object.keys() is sufficiently well supported.
  2. Make test titles more informative on jsonapi vs astra, currently printing StargateMongoose - [object Object] Connection
  3. Assert that TEST_DOC_DB is set to a valid value.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Changes manually tested
  • Automated Tests added/updated
  • Documentation added/updated
  • CLA Signed: DataStax CLA

@vkarpov15 vkarpov15 requested a review from a team as a code owner May 1, 2023 16:24
@kathirsvn
Copy link
Contributor

@vkarpov15 Good improvements!
There are merge conflicts. Also, I believe this PR will work only with latest JSON API (i.e. the version yet to be published). If so, I can work on a JSON API release today.

@vkarpov15
Copy link
Collaborator Author

Correct, this will only work on latest JSON API. I'll fix the merge conflicts later today.

@kathirsvn
Copy link
Contributor

@vkarpov15 I resolved the merge conflicts, updated the API version and also changed the JSON API port as required with the latest version (port has been changed from 8080 to 8181)

@vkarpov15
Copy link
Collaborator Author

LGTM, thanks @kathirsvn

1 similar comment
@vkarpov15
Copy link
Collaborator Author

LGTM, thanks @kathirsvn

@kathirsvn kathirsvn merged commit 1654405 into main May 2, 2023
@kathirsvn kathirsvn deleted the vkarpov15/deleteone-sort branch May 2, 2023 01:44
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