ChainSafe / ChainSafe/web3.unity

Unify `DataStorage`

Offen
#976 0 Kommentare 0 Reaktionen 1 zugewiesene Person Beansprucht von @rob1997 Auf GitHub ansehen
Type: Enhancement Type: Feature
Vorherrschende Sprache
C#
Sterne
944
Forks
245
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.