web3.unity icon indicating copy to clipboard operation
web3.unity copied to clipboard

Unify `DataStorage`

Open rob1997 opened this issue 1 year ago • 0 comments

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 |

rob1997 avatar May 27 '24 08:05 rob1997