ChainSafe / ChainSafe/web3.unity

Unify `DataStorage`

Abierto
#976 0 comentarios 0 reacciones 1 asignado Reclamado por @rob1997 Ver en GitHub
Type: Enhancement Type: Feature
Lenguaje dominante
C#
Estrellas
944
Forks
245
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

**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 |

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.