Differential D12805

[lib] introduce new device list function to replace keyserver device
ClosedPublic
Actions

Authored by varun on Jul 19 2024, 12:52 PM.

Tags

None

Referenced Files

	F2623043: D12805.id.diff
	Fri, Sep 6, 9:50 PM

	Unknown Object (File)
	Fri, Sep 6, 3:02 AM

	Unknown Object (File)
	Wed, Sep 4, 5:21 AM

	Unknown Object (File)
	Mon, Sep 2, 11:34 AM

	Unknown Object (File)
	Sun, Sep 1, 10:16 AM

	Unknown Object (File)
	Fri, Aug 30, 10:17 PM

	Unknown Object (File)
	Fri, Aug 30, 4:29 PM

	Unknown Object (File)
	Fri, Aug 30, 3:21 PM

View All 38 Files

Subscribers

Details

Reviewers

bartek
ashoat

Commits

rCOMMc65be4f84cff: [lib] introduce new device list function to replace keyserver device

Summary

we want to be able to replace an existing keyserver with a new one

Depends on D12734

Test Plan

later in the diff stack I successfully replace my existing keyserver device with a new one

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Jul 19 2024, 12:52 PM

Herald added a subscriber: tomek. · View Herald TranscriptJul 19 2024, 12:52 PM

Harbormaster completed remote builds in B30515: Diff 42550.Jul 19 2024, 1:09 PM

varun requested review of this revision.Jul 19 2024, 1:09 PM

varun added a child revision: D12809: [lib] add new selector and function to get own user's keyserver device ID.Jul 19 2024, 2:18 PM

bartek requested changes to this revision.Jul 22 2024, 1:36 AM

bartek added inline comments.

lib/shared/device-list-utils.js
236–240 ↗	(On Diff #42550)	What if the old device is not on the list, but the new one is not? I think we should continue adding it in that case. Otherwise, this function would return successfully without the device list being updated

This revision now requires changes to proceed.Jul 22 2024, 1:36 AM

address feedback

Harbormaster completed remote builds in B30681: Diff 42792.Jul 25 2024, 8:53 PM

bartek accepted this revision.Jul 26 2024, 1:56 AM

This revision is now accepted and ready to land.Jul 26 2024, 1:56 AM

Closed by commit rCOMMc65be4f84cff: [lib] introduce new device list function to replace keyserver device (authored by varun). · Explain WhyThu, Aug 22, 9:24 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMc65be4f84cff: [lib] introduce new device list function to replace keyserver device.

Revision Contents
Changeset List

Path

Size

lib/

shared/

device-list-utils.js

30 lines

Diff 43575

lib/shared/device-list-utils.js

Loading...