[lib] Add function to remove device from device list
ClosedPublic
Actions

Authored by bartek on May 29 2024, 9:48 AM.

Details

Reviewers

kamil

Commits

rCOMMa22a27c78679: [lib] Add function to remove device from device list

Summary

Analogical to addDeviceToDeviceList(), added a function that removes a device.

Depends on D12238

Test Plan

Unit tests added in the next diff

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.May 29 2024, 9:48 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMay 29 2024, 9:48 AM

bartek added a child revision: D12240: [lib] Add unit tests for functions modifying device list.May 29 2024, 9:49 AM

Harbormaster completed remote builds in B29287: Diff 40752.May 29 2024, 10:05 AM

bartek published this revision for review.May 29 2024, 10:12 AM

kamil accepted this revision.Jun 3 2024, 4:40 AM

kamil added inline comments.

lib/shared/device-list-utils.js
211	Just wondering, shouldn't we throw here? Is there any use-case where we might know that we're attempting to remove a non-existing device?

This revision is now accepted and ready to land.Jun 3 2024, 4:40 AM

bartek added inline comments.Jun 6 2024, 12:48 AM

lib/shared/device-list-utils.js
211	In theory we could throw, but I wanted to keep parity with adding device to list which is idempotent if device already exist.

Closed by commit rCOMMa22a27c78679: [lib] Add function to remove device from device list (authored by bartek). · Explain WhyJun 6 2024, 1:50 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMa22a27c78679: [lib] Add function to remove device from device list.

Revision Contents
Changeset List

Path

Size

lib/

shared/

device-list-utils.js

25 lines

Diff 40752

View Options

lib/shared/device-list-utils.js

[lib] Add function to remove device from device listClosedPublicActions