[native] Update `updateClientDBThreadStoreThreadInfos` to use `values(...)`
ClosedPublic
Actions

Authored by atul on Apr 25 2023, 1:17 PM.

Details

Reviewers

ginsu
ashoat

Commits

rCOMM71833b54df11: [native] Update `updateClientDBThreadStoreThreadInfos` to use `values(...)`

Summary

Got feedback on D7590 to make this change. I had made the changes locally but then accidentally stashed them instead of amending diff... this diff fixes it up.

Test Plan

Close reading, will run migration locally and make sure things work as expected.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Apr 25 2023, 1:17 PM

Herald added a subscriber: tomek. · View Herald TranscriptApr 25 2023, 1:17 PM

atul published this revision for review.Apr 25 2023, 1:17 PM

ashoat accepted this revision.Apr 25 2023, 1:21 PM

This revision is now accepted and ready to land.Apr 25 2023, 1:21 PM

This revision was landed with ongoing or failed builds.Apr 25 2023, 1:28 PM

Closed by commit rCOMM71833b54df11: [native] Update `updateClientDBThreadStoreThreadInfos` to use `values(...)` (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM71833b54df11: [native] Update `updateClientDBThreadStoreThreadInfos` to use `values(...)`.

Harbormaster completed remote builds in B18767: Diff 25712.Apr 25 2023, 1:38 PM

Revision Contents
Changeset List

Path

Size

native/

redux/

client-db-utils.js

5 lines

Diff 25714

View Options

native/redux/client-db-utils.js

[native] Update `updateClientDBThreadStoreThreadInfos` to use `values(...)`ClosedPublicActions