Stef Schenkelaars

Results 11 comments of Stef Schenkelaars

@swelther I did try it with sequential updates. They are also [enabled in the test case](https://github.com/brendon/acts_as_list/pull/372/files#diff-95189592e42644445faf233a308b5a3aR12) but it still crashes. So I've added the constraints on my database, everything went...

@brendon And what do think about this if you compare it to [ranked-model](https://github.com/mixonic/ranked-model)? It seems that in the way they handle the positions in the database, it will not have...

Hi there, it has been some time since there was some activity here but maybe we can get some discussion started again. I would like to 'fix' this by putting...

Ok, had a day off today so did some quick tests. Basically I removed all methods and callbacks and created 4 callbacks that should handle the big chunks of logic:...

Hi @brendon, **Transactions** I have to look into this but I think in general it shouldn't be too big of a problem. Basically if someone kills the transaction, everything is...

Well I of course don't use it with only fetching the ids. I use it in an API call which returns multiple averages and ids grouped by some attribute. So...

@Animesh-Ghosh I don't think adding a `.ruby-version` makes sense since there is a test matrix for multiple ruby versions: https://github.com/igorkasyanchuk/active_storage_validations/blob/7e864bace93aac6816fb8d72d736e274c85787be/.github/workflows/minitest.yml#L9-L13 It might also be interesting to combine this addition with...

It seems that the slowness is originating from Rails 6.1.4 (which actually switches the Marcel gem).

I still don't really like the passing of these options all the time but I'm ok if it works.

@igorkasyanchuk Agree with @gr8bit that it's good for a merge. I think you are the only one who can do that.