Paths

Table of Contentst

Differential D13729

[lib] Update usages of userHasDeviceList to acknowledge accountMissingStatus
Needs ReviewPublic
Actions

Authored by inka on Wed, Oct 16, 8:38 AM.

Tags

None

Referenced Files

	F2988329: D13729.id45228.diff
	Wed, Oct 16, 4:57 PM

	F2988289: D13729.id.diff
	Wed, Oct 16, 4:57 PM

	F2988279: D13729.diff
	Wed, Oct 16, 4:54 PM

Subscribers

Details

Reviewers

tomek
kamil

Summary

issue: ENG-9633

Test Plan

Tested that when a user is missing a device list the accountMissingStatus is set. Checked that lastChecked is updated with next requests but missingSince stays the same.

checked that if non-empty a device list is returned by identity, accountMissingStatus is removed.

checked that further requests for the same user are stopped for 20s.

checked that if a user that has a recent lastChecked is picked in search (triggering onSelectUserFromSearch which calls useUsersSupportThickThreads) they are not queried for

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

inka created this revision.Wed, Oct 16, 8:38 AM

Herald added a subscriber: ashoat. · View Herald TranscriptWed, Oct 16, 8:38 AM

inka added a parent revision: D13728: [lib] Mark missing users when fetching device lists.Wed, Oct 16, 8:39 AM

Harbormaster completed remote builds in B32272: Diff 45228.Wed, Oct 16, 9:02 AM

inka requested review of this revision.Wed, Oct 16, 9:02 AM

Revision Contents
Changeset List

Path

Size

lib/

hooks/

user-identities-hooks.js

7 lines

shared/

dm-ops/

12 lines

thread-utils.js

14 lines

Diff 45228

lib/hooks/user-identities-hooks.js

Loading...

lib/shared/dm-ops/dm-op-utils.js

Loading...

lib/shared/thread-utils.js

Loading...