Paths

Table of Contentst

Differential D9003

[client-backup] implement backup handler
ClosedPublic
Actions

Authored by kamil on Aug 29 2023, 2:52 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Sep 6, 3:41 PM

	Unknown Object (File)
	Fri, Sep 6, 2:15 PM

	Unknown Object (File)
	Fri, Sep 6, 1:37 PM

	Unknown Object (File)
	Thu, Sep 5, 1:46 PM

	Unknown Object (File)
	Thu, Sep 5, 12:13 PM

	Unknown Object (File)
	Thu, Sep 5, 11:22 AM

	Unknown Object (File)
	Tue, Aug 27, 2:02 PM

	Unknown Object (File)
	Mon, Aug 26, 12:33 PM

Subscribers

Details

Reviewers

michal
marcin
bartek
inka

Commits

rCOMM4ab2a5637e7a: [client-backup] implement backup handler

Summary

For now in UserData we store UserStore. Each time it changes we upload a new backup.

Depends on D9002

Test Plan

Test both successful and failed uploads.

image.png (789×391 px, 74 KB)

image.png (803×392 px, 87 KB)

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Aug 29 2023, 2:52 AM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 29 2023, 2:52 AM

kamil added a child revision: D9004: [client-backup] add restore test.Aug 29 2023, 2:54 AM

Harbormaster failed remote builds in B22206: Diff 30482!Aug 29 2023, 3:03 AM

kamil edited the summary of this revision. (Show Details)Aug 29 2023, 3:30 AM

kamil edited the test plan for this revision. (Show Details)

CI

Harbormaster failed remote builds in B22209: Diff 30485!Aug 29 2023, 3:46 AM

CI again

kamil published this revision for review.Aug 29 2023, 4:05 AM

Harbormaster completed remote builds in B22210: Diff 30486.Aug 29 2023, 4:06 AM

kamil added a child revision: D9010: [client-backup] add local settings to store.Aug 29 2023, 7:35 AM

kamil removed a child revision: D9004: [client-backup] add restore test.Aug 29 2023, 7:39 AM

kamil added a child revision: D9004: [client-backup] add restore test.Aug 29 2023, 7:41 AM

kamil removed a child revision: D9004: [client-backup] add restore test.Aug 29 2023, 7:51 AM

kamil added a child revision: D9004: [client-backup] add restore test.Aug 29 2023, 7:54 AM

kamil removed a child revision: D9004: [client-backup] add restore test.Aug 29 2023, 7:55 AM

kamil added a reviewer: inka.Aug 29 2023, 8:38 AM

michal added inline comments.Aug 29 2023, 10:34 AM

native/backup/backup-handler.js
38 ↗	(On Diff #30486)	What do you think?
54 ↗	(On Diff #30486)	Even if it's opt-in we probably don't need to display the alert on success

bartek added inline comments.Aug 29 2023, 1:13 PM

native/backup/backup-handler.js
54 ↗	(On Diff #30486)	We have so much alert spam already 😅

remove success info
rename some vars

kamil added inline comments.Sep 4 2023, 3:10 AM

native/backup/backup-handler.js
38 ↗	(On Diff #30486)	I'm not convinced, `sha256Hash` is more current hash, value from `AsyncStorage` is recent as it was computed previously 😅
54 ↗	(On Diff #30486)	okay, I wanted to check how frequent data is backed up - but we can always check this in backup service logs

kamil added a child revision: D9004: [client-backup] add restore test.Sep 4 2023, 3:14 AM

kamil removed a child revision: D9004: [client-backup] add restore test.Sep 4 2023, 3:14 AM

Harbormaster completed remote builds in B22367: Diff 30734.Sep 4 2023, 3:25 AM

michal accepted this revision.Sep 4 2023, 8:31 AM

michal added inline comments.

native/backup/backup-handler.js
38 ↗	(On Diff #30486)	What about `mostRecentlyUploadedBackupHash`? :P

This revision is now accepted and ready to land.Sep 4 2023, 8:31 AM

Closed by commit rCOMM4ab2a5637e7a: [client-backup] implement backup handler (authored by kamil). · Explain WhySep 5 2023, 5:13 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM4ab2a5637e7a: [client-backup] implement backup handler.

kamil mentioned this in D9080: [client-backup] rename `recentBackupHash` -> `mostRecentlyUploadedBackupHash`.Sep 5 2023, 5:14 AM

kamil mentioned this in rCOMMfc35759754a3: [client-backup] rename `recentBackupHash` -> `mostRecentlyUploadedBackupHash`.Sep 5 2023, 5:47 AM

Revision Contents
Changeset List

Path

Size

native/

backup/

backup-handler.js

60 lines

2 lines

2 lines

Diff 30485

native/backup/backup-handler.js

Loading...

native/backup/constants.js

Loading...

native/root.react.js

Loading...