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

Fix MONGOID-5060 #delete causes dependent associations to be destroyed #4962

Merged
merged 1 commit into from Mar 30, 2021

Conversation

p-mongo
Copy link
Contributor

@p-mongo p-mongo commented Mar 15, 2021

No description provided.

@p-mongo p-mongo force-pushed the 5060 branch 2 times, most recently from f9621bd to cc3c9e9 Compare March 30, 2021 00:28
@p-mongo p-mongo changed the title MONGOID-5060 5060 Fix MONGOID-5060 #delete causes dependent associations to be destroyed Mar 30, 2021
@p-mongo
Copy link
Contributor Author

p-mongo commented Mar 30, 2021

@p-mongo p-mongo force-pushed the 5060 branch 2 times, most recently from 3a69cbc to c1a3968 Compare March 30, 2021 02:07
# Deletes the album from the database
band.destroy

The behavior of Mongoid 7.3 is consistent with how ActiveRecord behaves.
Copy link
Collaborator

Choose a reason for hiding this comment

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

The behavior of Mongoid 7.3 is consistent with how ActiveRecord behaves.

The behavior of Mongoid 7.3 is consistent with how ActiveRecord associations behave.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I prefer to not link to AR docs when not necessary to avoid link rot as well as sending users away from our documentation.

@p-mongo p-mongo merged commit 2d613e0 into mongodb:master Mar 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants