Page MenuHomePhabricator
Differential D12239

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

Authored by bartek on May 29 2024, 9:48 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jun 19, 12:16 PM
Unknown Object (File)
Sun, Jun 16, 9:08 AM
Unknown Object (File)
Sat, Jun 15, 4:06 PM
Unknown Object (File)
Tue, Jun 11, 4:15 PM
Unknown Object (File)
Mon, Jun 10, 8:38 PM
Unknown Object (File)
Mon, Jun 10, 5:41 PM
Unknown Object (File)
Mon, Jun 10, 12:20 AM
Unknown Object (File)
Sun, Jun 9, 5:43 PM
View All 21 Files
Subscribers
ashoat
tomek

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
Lint Not Applicable
Unit
Tests Not Applicable

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.Mon, Jun 3, 4:40 AM
kamil added inline comments.
lib/shared/device-list-utils.js
211 ↗(On Diff #40752)

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.Mon, Jun 3, 4:40 AM
bartek added inline comments.Thu, Jun 6, 12:48 AM
lib/shared/device-list-utils.js
211 ↗(On Diff #40752)

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 WhyThu, Jun 6, 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

PathSize
lib/
shared/
25 lines

Diff 41027

View Options

lib/shared/device-list-utils.js

Loading...