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.

Sign up for GitHub

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

Jump to bottom

Get rid of recursive merging when using set() on nested paths #9121

Closed
vkarpov15 opened this issue Jun 11, 2020 · 0 comments
Closed

Get rid of recursive merging when using set() on nested paths #9121

vkarpov15 opened this issue Jun 11, 2020 · 0 comments
Assignees
@vkarpov15
Labels
backwards-breaking enhancement This issue is a user-facing general improvement that doesn't fix a bug or add a new feature
Milestone
6.0.0-rc0

Comments

@vkarpov15
Copy link
Collaborator

Re: #9046, recursive merging behavior causes a lot of confusion.
@vkarpov15 vkarpov15 added enhancement This issue is a user-facing general improvement that doesn't fix a bug or add a new feature backwards-breaking labels Jun 11, 2020
@vkarpov15 vkarpov15 added this to the 6.0 milestone Jun 11, 2020
@vkarpov15 vkarpov15 mentioned this issue Jul 19, 2020
Closed
IslandRhythms added a commit that referenced this issue Jul 6, 2021
@IslandRhythms
Merge branch '6.0' of https://github.com/Automattic/mongoose into gh-…
ba0b871 
…9121
@vkarpov15 vkarpov15 assigned vkarpov15 and unassigned IslandRhythms Jul 26, 2021
vkarpov15 added a commit that referenced this issue Jul 26, 2021
@vkarpov15
test: repro #9121
45fed75
vkarpov15 added a commit that referenced this issue Jul 26, 2021
@vkarpov15
BREAKING CHANGE: overwrite instead of merging when setting nested paths
98150ac 
Fix #9121
vkarpov15 added a commit that referenced this issue Jul 26, 2021
@vkarpov15
docs: update docs to reflect #9121
77fc985
vkarpov15 added a commit that referenced this issue Jul 26, 2021
@vkarpov15
refactor: clean up someCondition re: #9121
3a900de
@vkarpov15 vkarpov15 mentioned this issue Jun 21, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vkarpov15 vkarpov15

Labels
backwards-breaking enhancement This issue is a user-facing general improvement that doesn't fix a bug or add a new feature
Projects
None yet
Milestone
6.0.0-rc0
Development

No branches or pull requests
2 participants
@vkarpov15 @IslandRhythms