-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Refactored Timestampable model behavior. #14027
Conversation
Codecov Report
@@ Coverage Diff @@
## 4.0.x #14027 +/- ##
==========================================
+ Coverage 68.62% 68.62% +<.01%
==========================================
Files 470 470
Lines 95148 95158 +10
==========================================
+ Hits 65294 65301 +7
- Misses 29854 29857 +3
Continue to review full report at Codecov.
|
@SidRoberts Is it ready to merge |
N.B. Could you create a new issue for "Quick question" to allow other contributors a chance to speak up, and to take a fresh look at this issue later. |
@sergeyklay It is ready. |
@SidRoberts Could you please rebase |
Thank you @SidRoberts |
Hello!
In raising this pull request, I confirm the following:
Quick question: Why would anyone want to update multiple fields with the same timestamp? Could we remove this functionality?