Page MenuHomePhabricator

[native] Use processDBStoreOperations function everywhere
ClosedPublic

Authored by tomek on Tue, May 7, 8:31 AM.
Tags
None
Referenced Files
F1787674: D11922.diff
Sat, May 18, 10:16 PM
Unknown Object (File)
Fri, May 17, 5:07 PM
Unknown Object (File)
Fri, May 17, 10:16 AM
Unknown Object (File)
Fri, May 17, 2:40 AM
Unknown Object (File)
Mon, May 13, 1:57 PM
Unknown Object (File)
Sun, May 12, 1:33 PM
Unknown Object (File)
Sun, May 12, 11:38 AM
Unknown Object (File)
Sat, May 11, 7:44 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tomek requested review of this revision.Tue, May 7, 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.Wed, May 8, 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.