ChainSafe / ChainSafe/web3.unity

Unify `DataStorage`

Open
#976 0 comments 0 reactions 1 assignee Claimed by @rob1997 View on GitHub
Type: Enhancement Type: Feature
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.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.