ChainSafe / ChainSafe/web3.unity
Unify `DataStorage`
- Dominant language
- C#
- Stars
- 944
- Forks
- 245
- PR merge metrics
- No merged PRs in 30d
Description
**Summary**
Currently we've only one `DataStorage` for Wallet Connect but we can save all data instances via one `DataStorage`.
**Suggested features or ways to address the summary**
- We can have storable instances as and injectable `IStorable` with a `StoragePath` and `LoadOnInitialize` properties that can be used to save/load data on Web3 initialize.
- We can have `DataStorage` as an injectable service that can save/load data.
**Motivation**
- This will help us easily save and load data by just extending `IStorable`, will increase reusability, maintainability and readability of the codebase.
**Acceptance Criteria**
- Instances can be saved and loaded just by extending `IStorable`
- `DataStorage` will be responsible for saving, loading, deleting and resetting data
**Blockers**
(OPTIONAL) [Do you see any blocker that needs to be done before implementing this feature?]
**Dependencies**
(OPTIONAL) [Do you know about dependencies that directly affect this issue?]
**Additional Context**
(OPTIONAL) [Add any other context, files, images (screenshots/videos), diagrams, or documents about the feature request here.]
**Final Steps**
* Give it one of these labels `Type: Feature` `Type: Enhancement`
* Add your estimation
**Estimations**
| 3 | Mild efort | A day | Low complexity | Low |
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.