I've tested the new feature for processing records in batches, and I
noticed the each method does not work with named_scopes. I attempted
to make a simple patch, but added no tests yet.
I think it's a really simple change that can help developers a lot
saving time and making the code more DRY.
Not a blocker, but around I've seen to much custom solutions for
running plugin migrations.
I created a patch for this, I think it could work for the 95% of the
existing applications.
There's a new regression in RC2 from the autosave logic, on any object
with a has_one :through that's been :included (even if autosave is
off). Very small patch:
I've been having a lot of issues with the new autosave feature in 2.3
I could easily workaround most of these issues if I had the ability to
explicitly disable autosave for certain relationships in such a way
that would override the autosave that happens when the association is
a "new_record?"
I’ve been trying to keep up to speed on all the stuff going into 2.3, but I missed mention of “the new autosave feature”. Google only returns links to lighthouse tickets that aren’t very explanatory. Could someone link me to it?
I’ve been having a lot of issues with the new autosave feature in 2.3
I could easily workaround most of these issues if I had the ability to
explicitly disable autosave for certain relationships in such a way
that would override the autosave that happens when the association is
a “new_record?”
You don’t have any issues with the new autosave behaviour, rather with the behaviour which already existed. But we went over that already on the ticket.
Just wanted to add this to the list, DDLs in sqlite3 transactions (and
savepoints and version comparison bugfixes to-boot). Koz is already
looking at it - but thought I should mention it in here in case:
At the moment, if you put two layout calls with options into a
controller, then the options will be merged together, but the template
name will be last-in-wins. This can be confusing, and the docs don't
actually say not to put two layout calls in one controller, AND the
layout guide actually gives this as an example of usage.
Instead of fixing the guide, I thought it was a nice idea to have
multiple layout methods, so I wrote this patch - which also fixes the
kind-of bug of conditions being merged but layout names being
overwritten.