Paths

Table of Contentst

Differential D11835

[web] add `GetDeviceListsForUsers` in `IdentityServiceClient`
ClosedPublic
Actions

Authored by kamil on Apr 29 2024, 3:02 PM.

Tags

None

Referenced Files

None

Subscribers

Details

Reviewers

tomek
bartek

Commits

rCOMMfd94c324c7e0: [web] add `GetDeviceListsForUsers` in `IdentityServiceClient`

Summary

JS implementation of GetDeviceListsForUsers RPC

Depends on D11834

Test Plan

Call this method for multiple user and test if it works

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Apr 29 2024, 3:02 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptApr 29 2024, 3:02 PM

Harbormaster completed remote builds in B28580: Diff 39660.Apr 29 2024, 3:19 PM

kamil published this revision for review.Apr 29 2024, 3:51 PM

tomek accepted this revision.Apr 30 2024, 8:54 AM

tomek added inline comments.

web/grpc/identity-service-client-wrapper.js
548–556 ↗	(On Diff #39660)	This can be simplified using `_.mapValues` - but up to you.

This revision is now accepted and ready to land.Apr 30 2024, 8:54 AM

kamil added inline comments.Apr 30 2024, 9:57 AM

web/grpc/identity-service-client-wrapper.js
548–556 ↗	(On Diff #39660)	We used a similar approach in this file (and the native version of it) so I prefer to stick to it. Also, my personal opinion is that with `mapValues` we gain a bit shorter code but a lot less readable.

rebase before landing

Harbormaster completed remote builds in B28598: Diff 39686.Apr 30 2024, 10:13 AM

Closed by commit rCOMMfd94c324c7e0: [web] add `GetDeviceListsForUsers` in `IdentityServiceClient`. · Explain WhyApr 30 2024, 10:23 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMMfd94c324c7e0: [web] add `GetDeviceListsForUsers` in `IdentityServiceClient`.

Revision Contents
Changeset List

Path

Size

lib/

types/

identity-service-types.js

2 lines

web/

grpc/

identity-service-client-wrapper.js

28 lines

identity-service-context-provider.react.js

1 line

Diff 39690

lib/types/identity-service-types.js

Loading...

web/grpc/identity-service-client-wrapper.js

Loading...

web/grpc/identity-service-context-provider.react.js

Loading...