Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: make storage-wrapper operations synchronous #12913

Open
2 of 9 tasks
Tracked by #10784
joaoloureirop opened this issue Jan 9, 2025 · 0 comments
Open
2 of 9 tasks
Tracked by #10784

refactor: make storage-wrapper operations synchronous #12913

joaoloureirop opened this issue Jan 9, 2025 · 0 comments

Comments

@joaoloureirop
Copy link
Contributor

joaoloureirop commented Jan 9, 2025

What is this about?

Now that we use MMKV, device storage operations are synchronous.

Our storage-wrapper methods are async only to support E2E, causing the whole storage-wrapper class to be behind a promise based architecture.

We need to decouple E2E fallbacks from the storage-wrapper implementation to have properly typed mmkv operations.

Scenario

N/A

Design

N/A

Technical Details

PR feat: Enable feature Flag Values with Scope Based on "threshold" in mobile has a draft commit on how it expects storage-wrapper interface to be.

Threat Modeling Framework

N/A

Acceptance Criteria

storage-wrapper methods to be sync when not running on E2E mode

Stakeholder review needed before the work gets merged

  • Engineering (needed in most cases)
  • Design
  • Product
  • QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
  • Security
  • Legal
  • Marketing
  • Management (please specify)
  • Other (please specify)

References

Epic: #10784

@joaoloureirop joaoloureirop changed the title refactor: make storage wrapper operation synchronous refactor: make storage wrapper operations synchronous Jan 9, 2025
@joaoloureirop joaoloureirop changed the title refactor: make storage wrapper operations synchronous refactor: make storage-wrapper operations synchronous Jan 9, 2025
@joaoloureirop joaoloureirop removed their assignment Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant