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

7.4 #13613

Merged
merged 56 commits into from
Jul 18, 2023
Merged

7.4 #13613

merged 56 commits into from
Jul 18, 2023

Conversation

vkarpov15
Copy link
Collaborator

Summary

Examples

hasezoey and others added 30 commits June 16, 2023 12:44
…ion, allow querying subdocuments with defaults disabled

Fix #13512
types: augment bson.ObjectId instead of adding on own type
…karpov15/schema-collection-options

Revert "Revert "feat(schema): add collectionOptions option to schemas""
feat(query): delay converting documents into POJOs until query execution, allow querying subdocuments with defaults disabled
fix(document): allow setting keys with dots in mixed paths underneath nested paths
…r, add `createClient()` method to handle creating MongoClient
Move all MongoDB-specific connection logic into driver layer, add `createClient()` method to handle creating MongoClient
IslandRhythms and others added 19 commits July 5, 2023 11:23
feat(connection): add `Connection.prototype.removeDb()` for removing a related connection
feat: upgrade to MongoDB Node.js driver 5.7.0
feat(model): add create option "immediateError"
feat: support generating custom cast error message with a function
Copy link
Collaborator

@hasezoey hasezoey left a comment

Choose a reason for hiding this comment

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

LGTM, works with current latest typegoose without problems

though some things to note:

  • all Schema.Types.*.setters properties seem to not have any jsdoc associated with them

Copy link
Collaborator

@AbdelrahmanHafez AbdelrahmanHafez left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@vkarpov15 vkarpov15 merged commit 52a6485 into master Jul 18, 2023
23 checks passed
@vkarpov15 vkarpov15 deleted the 7.4 branch July 18, 2023 21:08
@vkarpov15 vkarpov15 restored the 7.4 branch July 18, 2023 21:08
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.

5 participants