-
-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Issue14204 #14212
Issue14204 #14212
Conversation
Issue #14204 unnecessary types removed from QueryOptions interface in query.d.ts
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.
This PR is way off base, and #14228 fixes this issue anyway, so I will close this
@@ -0,0 +1,14 @@ | |||
import * as mongoose from "mongoose"; |
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.
These changes are irrelevant and unnecessary
@@ -99,82 +99,10 @@ declare module 'mongoose' { | |||
PopulateOption, | |||
SessionOption { | |||
arrayFilters?: { [key: string]: any }[]; | |||
batchSize?: number; |
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.
What makes you think that deleting all the query options that operations other than updateOne()
use is a valid solution?
@@ -0,0 +1,46 @@ | |||
const mongoose = require('mongoose'); |
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.
This isn't relevant to the issue at hand, please delete
Summary
Examples