We're using the redux/state management as a sudo event system. It would be much better to actually use a proper pub/sub mechanism, preferably one native to node/hermes. There is the standard node.js Event system which I believe is supported by Hermes and also one built into RN (a little thin on documentation).
This PR needs to include:
[ ] Integrating / wrapping / whatever is needed to use. one of the two event systems noted in the description above;
[ ] Replace existing usage of the state management system with an event system. Known locations include:
[ ] Pin-changed in Settings.
[ ] addCredentialPressed in credential list screen (BC Wallet only)
jleach
commented
1 year ago
We're using the redux/state management as a sudo event system. It would be much better to actually use a proper pub/sub mechanism, preferably one native to node/hermes. There is the standard node.js Event system which I believe is supported by Hermes and also one built into RN (a little thin on documentation).
This PR needs to include: