Skip to content

Update mongodb to ^3.6.4 #618

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
Feb 17, 2021
Merged

Update mongodb to ^3.6.4 #618

merged 1 commit into from
Feb 17, 2021

Conversation

wolrajhti
Copy link
Contributor

Signed-off-by: wolrajhti [email protected]

Since 3.6.4 mongodb driver console.warn over and over a depreciation message related to the new format for WriteConcern options. This PR add support for new format, while keeping backward compatibility.

(There is an opened issue for the mongodb driver, https://jira.mongodb.org/browse/NODE-3114, so this PR alone will not disable all ##console.warn).

Related issue

#616

Checklist

  • DCO (Developer Certificate of Origin) signed in all commits
  • npm test passes on your machine
  • New tests added or existing tests modified to cover all changes
  • Code conforms with the style guide
  • Commit messages are following our guidelines

Signed-off-by: wolrajhti <[email protected]>
Copy link
Contributor

@agnes512 agnes512 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR ❤️ I saw the issue that day, but I was not able to stop it outputting warnings :( But still, thanks for adding support for new format!

@agnes512 agnes512 merged commit c7b840e into loopbackio:master Feb 17, 2021
@wolrajhti
Copy link
Contributor Author

Hi

I just submit a PR to node-mongodb-native driver to fix the merging of writeConcern mongodb/node-mongodb-native#2744 which stop warning outputs.

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