- remove SQL
- use update_column instead SQL ( avoid transaction and callbacks)
- extract condition to method
- use increment_counter instead SQL
Looks like it’s failing? Any idea why?
I remove now_date accidentally . It may work now.
It’s still failing - I tried locally too outside of travis and unit tests are failing. Are the unit tests not working locally for you?
fixed by now
I merged this, but squashed your commits first. Looks good, thanks 