VEuPathDB / web-monorepo

A monorepo that contains all frontend code for VEuPathDB websites
Apache License 2.0
2 stars 0 forks source link

Remove searchTree (derived state) from redux store #1184

Closed dmfalke closed 3 months ago

dmfalke commented 3 months ago

Derived state does not belong in the redux store. Not only is it an anti-pattern, it is also causing a delay in some rendering since it is dependent on static/all-data-loaded, which depends on other resources being loaded.

See https://github.com/VEuPathDB/web-monorepo/blob/501d0bbce2e850358c7ccf134068c85fb9c8d1a4/packages/libs/web-common/src/wrapStoreModules.js#L89-L93. Turn this transformation into a selector function.