Page MenuHomePhabricator
Differential D14000

[CommCoreModule][native_rust_library] move constants to one place
ClosedPublic
Actions

Authored by kamil on Nov 22 2024, 2:15 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 1:38 PM
Unknown Object (File)
Mon, Dec 16, 12:55 PM
Unknown Object (File)
Mon, Dec 16, 2:42 AM
Unknown Object (File)
Mon, Dec 9, 10:46 AM
Unknown Object (File)
Thu, Dec 5, 2:52 PM
Unknown Object (File)
Thu, Dec 5, 12:46 AM
Unknown Object (File)
Wed, Dec 4, 11:08 PM
Unknown Object (File)
Sat, Nov 30, 9:56 PM
View All 21 Files
Subscribers
ashoat

Details

Reviewers
bartek
tomek
Commits
rCOMM5a5a75ae2939: [CommCoreModule][native_rust_library] move constants to one place
Summary

Part of ENG-9656

I'm not sure why it was previously defined in two places. It makes it hard to maintain, so I'm cleaning this up. I added the missing comment.

Depends on D13937

Test Plan

Tested creating backup and secondary device login

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Nov 22 2024, 2:15 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptNov 22 2024, 2:15 AM
kamil added a child revision: D14001: [backup-client] set `backupID` after uploading User Keys.Nov 22 2024, 2:19 AM
Harbormaster failed remote builds in B32761: Diff 45937!Nov 22 2024, 2:31 AM
kamil updated this revision to Diff 45940.Nov 22 2024, 2:42 AM

build .wasm

Harbormaster completed remote builds in B32764: Diff 45940.Nov 22 2024, 3:06 AM
kamil edited the summary of this revision. (Show Details)Nov 22 2024, 3:56 AM
kamil published this revision for review.Nov 22 2024, 4:01 AM
bartek accepted this revision.Sun, Nov 24, 11:20 PM
This revision is now accepted and ready to land.Sun, Nov 24, 11:20 PM
kamil updated this revision to Diff 45992.Mon, Nov 25, 3:38 AM

rebase before landing

Harbormaster completed remote builds in B32805: Diff 45992.Mon, Nov 25, 3:58 AM
Closed by commit rCOMM5a5a75ae2939: [CommCoreModule][native_rust_library] move constants to one place (authored by kamil). · Explain WhyMon, Nov 25, 4:07 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM5a5a75ae2939: [CommCoreModule][native_rust_library] move constants to one place.

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
DatabaseManagers/
2 lines
16 lines
Tools/
4 lines
native_rust_library/
src/
1 line
web/
shared-worker/
_generated/

Diff 45940

View Options

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.h

Loading...
View Options

native/cpp/CommonCpp/DatabaseManagers/SQLiteQueryExecutor.cpp

Loading...
View Options

native/cpp/CommonCpp/Tools/CommSecureStore.h

Loading...
View Options

native/native_rust_library/src/constants.rs

Loading...
View Options

web/shared-worker/_generated/comm_query_executor.wasm

Loading...