HomePhabricator
Diffusion Comm b322cf848bb5

[native] Remove dangling device after QR auth failure

Description

[native] Remove dangling device after QR auth failure

Summary:
Address ENG-10309.
Try reverting device list update after the process failed at a later point.

Test Plan:

  1. Replaced await sendDeviceListUpdateSuccessMessage() with throw new Error('foo')
  2. After alert is shown, verified that:
    • No new device is present on the Linked Devices screen
    • Checked DynamoDB - two device list updates are present: the first update added a new device and the second update removed it. Before this diff, only the former update was present, resulting in dangling device on the list.

Reviewers: tomek, kamil, ashoat

Reviewed By: ashoat

Subscribers: ashoat

Differential Revision: https://phab.comm.dev/D14431

Details

Provenance
bartekAuthored on Wed, Mar 5, 1:41 PM
Reviewer
ashoat
Differential Revision
D14431: [native] Remove dangling device after QR auth failure
Parents
rCOMMf9c41e3a3d11: [native] update Podfile.lock
Branches
Loading...
Tags
Loading...