Jesper Josefsson
Jesper Josefsson
I’d say it’s probably a bug. I’ll investigate. > 28 nov. 2018 kl. 03:39 skrev Brice Vallieres : > > I'm running an upsert that replaces the current value (e.g....
@bvallier I'm having trouble reproducing this, can you write a failing test?
This spec should demonstrate that setting stuff to nil is possible on some level: https://github.com/jesjos/active_record_upsert/blob/master/spec/active_record/base_spec.rb#L66 I think maybe something more subtle is going on. Still investigating.
Oh, I'm sorry. That test runs green b/c it uses `attributes: [:id, :name]`.
I have this vague memory that we added callback support in order to have timestamps work like you expect, but I'm not sure. I'm all for removing callback support if...
Seems this was fixed in 5.1.2 but 5.1.1 is giving me errors.
It's not currently possible with this library, but please do provide a PR that adds that functionality
Hi, I was looking into this and couldn't find any good seam where we could insert a check. We can introspect the result of the SQL query, so in theory...
PR welcome, but the maintainers won't develop this functionality.
@andrewclink Can you rebase your branch on the target branch?