Skip to content

Make drivers return selectors instead of storing values in views (fix #1247)

Nicolas Frandeboeuf requested to merge nicofrand/do-not-store-values-in-1247 into main

I did not see any issue while testing everything, I checked the memoization by adding console.log in the functions called and navigating across charts/budgets/reports/etc.

memoization is mainly based on state.banks, which is kind of identical to the previous way (we called driver.getView every time the driver or state.banks dependency changed).

Merge request reports

Loading