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

Update README.md #13534

Closed
wants to merge 1 commit into from
Closed

Update README.md #13534

wants to merge 1 commit into from

Conversation

kakopappa
Copy link

While upgrading from mongoose 6.x to 7.x i noticed mongoose.Schema.ObjectId has been changed to mongoose.Schema.Types.ObjectId so update the readme to reflect this change

Verified

This commit was signed with the committer’s verified signature.
albertvillanova Albert Villanova del Moral
Looks like mongoose.Schema.ObjectId has been changed to mongoose.Schema.Types.ObjectId
@vkarpov15
Copy link
Collaborator

mongoose.Schema.ObjectId was not changed to mongoose.Types.ObjectId . Those are two distinct classes. What makes you think that we removed mongoose.Schema.ObjectId?

@kakopappa
Copy link
Author

kakopappa commented Jun 21, 2023

Hi @vkarpov15

The reason is VSCode did not recognize mongoose.Schema.ObjectId as a class however mongoose.Schema.Types.ObjectId did. so I got confused

image

great project. sorry about the confusion.

@kakopappa kakopappa closed this Jun 21, 2023
@vkarpov15 vkarpov15 reopened this Jun 23, 2023
@vkarpov15
Copy link
Collaborator

It looks like our TypeScript type defs don't have mongoose.Schema.ObjectId, so it may be our fault. We will keep this open to check.

@vkarpov15 vkarpov15 added this to the 7.3.2 milestone Jun 23, 2023
vkarpov15 added a commit that referenced this pull request Jun 25, 2023

Verified

This commit was signed with the committer’s verified signature.
albertvillanova Albert Villanova del Moral
…ma.Types.ObjectId`

Fix #13534
@vkarpov15
Copy link
Collaborator

Closing in favor of #13543

@vkarpov15 vkarpov15 closed this Jun 25, 2023
vkarpov15 added a commit that referenced this pull request Jun 26, 2023

Verified

This commit was signed with the committer’s verified signature.
albertvillanova Albert Villanova del Moral
types: support `mongoose.Schema.ObjectId` as alias for `mongoose.Schema.Types.ObjectId`
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.

None yet

2 participants