Pavlo Tsimura
Pavlo Tsimura
Found and reported another bad `mergeCollection` issue: https://github.com/Expensify/react-native-onyx/issues/514
> How is the QA coming on the original PR and what is the status/ETA of getting this merged? This PR unfortunately must be held for the other Onyx issues...
@blazejkustra is there any chance you could help me with reviewing this TS-wise? 🙏 There are a couple of tough places I'm not entirely sure how to implement since the...
> Would it be possible to wait with this PR until the migration is merged? Absolutely 👍
> Update: [OnyxUtils](https://github.com/Expensify/react-native-onyx/pull/523#pullrequestreview-1996416637) was merged yesterday, we can remove HOLD from the title! Thanks @blazejkustra. That PR caused one blocking regression, reported here: https://github.com/Expensify/react-native-onyx/pull/523/files#r1568401082
## Proposal ### Please re-state the problem that we are trying to solve in this issue. When editing a Distance request, the duplicate waypoints error is not shown immediately. ###...
This caused a regression: https://github.com/Expensify/App/issues/34385
This will be fixed in https://github.com/Expensify/App/issues/34988#issuecomment-1906783247
From my side, I can confirm it's fixed: https://github.com/Expensify/App/assets/12595293/90324d6b-b42d-4006-aee2-6370d52a6bf2
First of all, this is not a deploy blocker – it's happening in prod as well: We're potentially facing the **out-of-order Onyx updates** again: When coming back online, 2 conflicting...