ChainSafe / ChainSafe/web3.unity

Unify `DataStorage`

オープン
#976 コメント 0 件 リアクション 0 件 担当者 1 名 @rob1997 が担当を希望しています GitHub で見る
Type: Enhancement Type: Feature
主要言語
C#
スター
944
フォーク
245
PR マージ指標
30日以内にマージされた PR はありません

説明

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

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。