[native] regenerate backup on every password change
ClosedPublic
Actions

Authored by kamil on Thu, Nov 28, 6:23 AM.

Details

Reviewers

bartek
tomek

Commits

rCOMM863c17c11b6c: [native] regenerate backup on every password change

Summary

ENG-8070

Some concerns are listed the comment in ENG-8070.

Depends on D14063

Test Plan

On primary device, change password and check that new backup was uploaded

Diff Detail

Repository

rCOMM Comm

Branch

land

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Thu, Nov 28, 6:23 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptThu, Nov 28, 6:23 AM

kamil edited the summary of this revision. (Show Details)Thu, Nov 28, 6:25 AM

kamil added inline comments.

native/backup/backup-handler.js
149 ↗	(On Diff #46109)	This should be added to diff earlier in the stack, I am going to fix this in next rebase

Harbormaster completed remote builds in B32893: Diff 46109.Thu, Nov 28, 6:41 AM

kamil published this revision for review.Thu, Nov 28, 9:10 AM

bartek accepted this revision.Tue, Dec 3, 12:29 AM

This revision is now accepted and ready to land.Tue, Dec 3, 12:29 AM

rebase before landing

Harbormaster failed remote builds in B32920: Diff 46181!Tue, Dec 3, 4:38 AM

rebase before landing v2

Harbormaster completed remote builds in B32922: Diff 46186.Tue, Dec 3, 5:58 AM

Closed by commit rCOMM863c17c11b6c: [native] regenerate backup on every password change (authored by kamil). · Explain WhyTue, Dec 3, 6:28 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM863c17c11b6c: [native] regenerate backup on every password change.

Revision Contents
Changeset List

Path

Size

lib/

reducers/

backup-reducer.js

5 lines

Diff 46181

View Options

lib/reducers/backup-reducer.js

[native] regenerate backup on every password changeClosedPublicActions