ChainSafe / ChainSafe/web3.unity
Unify `DataStorage`
- Lingua principale
- C#
- Stelle
- 944
- Fork
- 245
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
**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 |
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Valutazione
Questa issue non è ancora stata valutata.