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

Rails minor version with 667 #5

Closed
wants to merge 3 commits into from
Closed

Conversation

a2ikm
Copy link
Owner

@a2ikm a2ikm commented Dec 14, 2020

This PR adds commits of mongomapper#667 on mongomapper#673.

a2ikm and others added 3 commits December 14, 2020 18:33
* ActiveRecord updates its changes/previous_changes before
  after_create/after_update callbacks
* However, MongoMapper does that after the callbacks, therefore the
  changes/previous_changes values don't match with equivalent code in
  ActiveRecord.
* Also cover the case when some callbacks didn't get called
@a2ikm a2ikm changed the base branch from rails-minor-version to master December 14, 2020 10:08
@a2ikm a2ikm force-pushed the rails-minor-version-with-667 branch from 1d06e2c to 2a69c8d Compare December 14, 2020 10:08
@a2ikm a2ikm closed this Apr 21, 2022
@a2ikm a2ikm deleted the rails-minor-version-with-667 branch April 21, 2022 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants