Page MenuHomePhabricator

[native] Use processDBStoreOperations function everywhere
ClosedPublic

Authored by tomek on May 7 2024, 8:31 AM.
Tags
None
Referenced Files
F3306123: D11922.id39991.diff
Mon, Nov 18, 6:27 PM
Unknown Object (File)
Sun, Nov 10, 2:59 PM
Unknown Object (File)
Sun, Nov 10, 1:14 PM
Unknown Object (File)
Sun, Nov 10, 11:14 AM
Unknown Object (File)
Sun, Nov 10, 4:17 AM
Unknown Object (File)
Sun, Oct 20, 6:46 PM
Unknown Object (File)
Oct 10 2024, 10:39 PM
Unknown Object (File)
Oct 10 2024, 10:39 PM
Subscribers

Details

Summary

We don't need to use async functions that process a single store. They will be deleted in the next diff.

https://linear.app/comm/issue/ENG-6004/wrap-all-store-operations-from-one-action-in-transaction

Depends on D11921

Test Plan

Checked if persisting drafts still works.

Diff Detail

Repository
rCOMM Comm
Branch
transactions
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

tomek requested review of this revision.May 7 2024, 9:00 AM

Checked if persisting drafts still works.

Why only drafts? We can probably test other stores by making changes in the app, closing keyserver connection and restarting it afterwards. I would also run sth like git grep "commCoreModule.process" before and after applying this diff.

This revision is now accepted and ready to land.May 8 2024, 3:18 AM

Checked if persisting drafts still works.

Why only drafts? We can probably test other stores by making changes in the app, closing keyserver connection and restarting it afterwards. I would also run sth like git grep "commCoreModule.process" before and after applying this diff.

Testing drafts is the easiest, and testing all might take too much time. Going to test e.g. messages and threads too.

As for grepping, in the next diff the functions are removed which effectively verifies if all the usages were replaced.