-
Notifications
You must be signed in to change notification settings - Fork 60
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
Mark soft deleted records as persisted. #2
Comments
Merged in ea986ac. |
I see a side effect here.
See: mongoid/mongoid@34e895ab The explanation provided by visnup:
|
This was merged because of mongoid internal changes. I'll take a look what changed, but without this, there were some issues. |
This should really be reverted imho, because it can and will break existing apps using mongoid-paranoia when mongoid @simi You should fix your problems in your specific app, I can help you :) |
Revert = red CI. Feel free to revert and make it green. I have not changed tests except one. If i remember well, this test will not pass 11ddc35. I'm around to help you. IRC Freenode, name |
I had a lot of issues today with this flag. CarrierWave photos were deleted and embeded models returned
persisted? #=> false
andto_param => nil
, which caused a lot of issued in rails url helpers.I started work in persistence branch.
Anyone against this change?
The text was updated successfully, but these errors were encountered: