Conditionally use default persist storage API or database depending on whether database is supported or not.
Depends on D7666
Details
Details
Switch to commReduxStorageEngine and test if it works for not supported database
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
Comment Actions
Wondering if this is the best solution. How about having two separate storage engines, one with only db and one with redux storage? Then we could conditionally set one or the other, but each of them would be a lot simpler.
| web/redux/comm-redux-storage-engine.js | ||
|---|---|---|
| 3 | Is this the right import? Do we have to include index in the path? | |
| web/redux/comm-redux-storage-engine.js | ||
|---|---|---|
| 3 | We have an ESLint rule that forces this. It can be ignored, but generally prefer to use index.js instead. This is the same import path we use in web/root.js currently | |