Results 367 comments of Niccolò Belli

I've added a regression test to prevent future occurrences.

You could also train it with the data from the MikroORM Discord/Slack I guess.

Could you please check if it somehow helps with https://github.com/vuejs/core/issues/12110 Thanks

I'd like to cherry-pick both this and https://github.com/vuejs/core/pull/12059 but I get conflicts with https://github.com/vuejs/core/pull/12059/files#diff-ddb3174016350dd4464764b4a1ba232d09ad1e75164d4a37758c759d019202e4 How would you solve it in the context of the other PR?

I guess that https://github.com/vuejs/core/pull/12059 needs to be rebased on top of this to fix the remaining issue, right?

@mmaietta thanks, I'm still interested to see this implemented. I have applied the changes you suggested among other changes I've copied from the riscv PR but I haven't been able...

Hi @mmaietta , no I'm sorry I still didn't have the chance to test it :(

>> looking at the implementation, it feels like it cannot really work correctly with the various options, namely I can't see any place where we would actually try to handle...

> If it behaves like that, it's most likely a bug, the method shouldn't change the collection contents if it's already loaded. That makes more sense now, but you will...

> That's exactly the user's problem, a misusage if you want. You should not use loadItems if you want to have new values regardless of the loaded state, just use...