Page MenuHomePhabricator
Differential D13397

[keyserver] new responder function to fetch all community infos with names
ClosedPublic
Actions

Authored by varun on Sep 20 2024, 1:23 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 28, 5:05 AM
Unknown Object (File)
Sat, Dec 28, 5:05 AM
Unknown Object (File)
Sat, Dec 28, 5:05 AM
Unknown Object (File)
Sat, Dec 28, 5:05 AM
Unknown Object (File)
Sat, Dec 28, 5:05 AM
Unknown Object (File)
Sat, Dec 28, 5:04 AM
Unknown Object (File)
Nov 20 2024, 7:03 AM
Unknown Object (File)
Nov 20 2024, 7:03 AM
View All 48 Files
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMMe9076b23f303: [keyserver] new responder function to fetch all community infos with names
Summary

returns an object instead of an array and checks if the viewer is logged in

Depends on D13396

Test Plan

tested at end of stack by calling responder successfullly from web

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun created this revision.Sep 20 2024, 1:23 AM
varun added a child revision: D13398: [lib][keyserver] new endpoint to get all community infos with names.
Harbormaster completed remote builds in B31725: Diff 44355.Sep 20 2024, 1:39 AM
varun requested review of this revision.Sep 20 2024, 1:39 AM
ashoat requested changes to this revision.Sep 20 2024, 5:40 AM
ashoat added inline comments.
keyserver/src/responders/community-responders.js
29 ↗(On Diff #44355)

I don't see this type defined anywhere, either on master or in this stack

This revision now requires changes to proceed.Sep 20 2024, 5:40 AM
varun updated this revision to Diff 44443.Sep 23 2024, 10:17 AM

D13396 contains the missing type in latest revision

varun updated this revision to Diff 44445.Sep 23 2024, 10:22 AM

makes more sense to have the FetchAllCommunityInfosWithNamesResponse in this diff instead of the previous

Harbormaster completed remote builds in B31783: Diff 44445.Sep 23 2024, 10:38 AM
Harbormaster completed remote builds in B31781: Diff 44443.Sep 23 2024, 10:50 AM
ashoat accepted this revision.Sep 24 2024, 9:43 AM
This revision is now accepted and ready to land.Sep 24 2024, 9:43 AM
Closed by commit rCOMMe9076b23f303: [keyserver] new responder function to fetch all community infos with names (authored by varun). · Explain WhySep 24 2024, 10:57 AM
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMMe9076b23f303: [keyserver] new responder function to fetch all community infos with names.

Revision Contents
Changeset List

PathSize
keyserver/
src/
responders/
27 lines

Diff 44443

View Options

keyserver/src/responders/community-responders.js

Loading...