Coming from #71202.
In order to simplify useOnyx API (and consequently Onyx) let's work on proposing the removal of some configurations we think are anti-patterns / unnecessary at this point.
The allowStaleData configuration controls whether the data will be retrieved from cache during the first render even if there is a pending merge for the key.
This, however, looks more like an anti-pattern and prone to bugs (we should always rely on updated data) and there are only 8 usages in the entire E/App codebase. I think we can remove this property and treat these cases on E/App with more suited solutions.
Let's investigate how we can achieve E/App needs without this prop and create a proposal/plan to remove it.
Issue Owner
Current Issue Owner: @mallenexpensify
Coming from #71202.
In order to simplify
useOnyxAPI (and consequently Onyx) let's work on proposing the removal of some configurations we think are anti-patterns / unnecessary at this point.The
allowStaleDataconfiguration controls whether the data will be retrieved from cache during the first render even if there is a pending merge for the key.This, however, looks more like an anti-pattern and prone to bugs (we should always rely on updated data) and there are only 8 usages in the entire E/App codebase. I think we can remove this property and treat these cases on E/App with more suited solutions.
Let's investigate how we can achieve E/App needs without this prop and create a proposal/plan to remove it.
Issue Owner
Current Issue Owner: @mallenexpensify