Page MenuHomePhabricator
Differential D13840

[backup-client] make `backupID` mandatory when restoring backup
ClosedPublic
Actions

Authored by kamil on Oct 31 2024, 5:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 29, 4:56 AM
Unknown Object (File)
Thu, Nov 28, 11:09 PM
Unknown Object (File)
Thu, Nov 28, 10:15 PM
Unknown Object (File)
Thu, Nov 28, 5:12 PM
Unknown Object (File)
Thu, Nov 28, 3:08 PM
Unknown Object (File)
Thu, Nov 28, 3:05 PM
Unknown Object (File)
Thu, Nov 28, 7:03 AM
Unknown Object (File)
Sat, Nov 23, 6:37 PM
View All 41 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
bartek
Commits
rCOMM955ec0a26dea: [backup-client] make `backupID` mandatory when restoring backup
Summary

ENG-6145.

Alternative, was to pass username/wallet to restoreBackup - but each time using this we should already have latestBackupInfo as it's needed during restore, so this better matches out protocol.

Depends on D13839

Test Plan

Tested in D13849

Diff Detail

Repository
rCOMM Comm
Branch
backup-work-5
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
native/
backup/
43 lines
cpp/
CommonCpp/
NativeModules/
3 lines
7 lines
_generated/
native_rust_library/
src/
11 lines
schema/
6 lines

Diff 45508

View Options

native/backup/use-client-backup.js

Loading...
View Options

native/cpp/CommonCpp/NativeModules/CommCoreModule.h

Loading...
View Options

native/cpp/CommonCpp/NativeModules/CommCoreModule.cpp

Loading...
View Options

native/cpp/CommonCpp/_generated/commJSI-generated.cpp

Loading...
View Options

native/cpp/CommonCpp/_generated/commJSI.h

Loading...
View Options

native/native_rust_library/src/backup.rs

Loading...
View Options

native/schema/CommCoreModuleSchema.js

Loading...