Page MenuHomePhabricator
Differential D13396

[keyserver] new fetcher to get all communities
ClosedPublic
Actions

Authored by varun on Sep 19 2024, 11:37 PM.
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)
Wed, Dec 11, 8:30 AM
Unknown Object (File)
Nov 20 2024, 7:03 AM
View All 61 Files
Subscribers
None

Details

Reviewers
ashoat
tomek
Commits
rCOMMe3c5f6ce1930: [keyserver] new fetcher to get all communities
Summary

we need the community id, name, and farcaster channel id for all communities so we can display them to users in a modal.

part of https://linear.app/comm/issue/ENG-9216/new-keyserver-endpoint-to-get-a-list-of-communities

Test Plan

tested the SQL query directly on my local mariadb instance. will test further in subsequent diffs that call the keyserver endpoint that returns this data

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

varun created this revision.Sep 19 2024, 11:37 PM
Harbormaster completed remote builds in B31724: Diff 44354.Sep 19 2024, 11:54 PM
varun requested review of this revision.Sep 19 2024, 11:54 PM
varun added a child revision: D13397: [keyserver] new responder function to fetch all community infos with names.Sep 20 2024, 1:23 AM
ashoat accepted this revision.Sep 20 2024, 5:41 AM
ashoat added inline comments.
keyserver/src/fetchers/community-fetchers.js
75 ↗(On Diff #44354)

Please keep all lines to a max of 80 chars width

This revision is now accepted and ready to land.Sep 20 2024, 5:41 AM
varun updated this revision to Diff 44442.Sep 23 2024, 10:15 AM

fix line length

varun updated this revision to Diff 44444.Sep 23 2024, 10:20 AM

move type to next diff

Harbormaster completed remote builds in B31780: Diff 44442.Sep 23 2024, 10:33 AM
Harbormaster completed remote builds in B31782: Diff 44444.Sep 23 2024, 10:37 AM
Closed by commit rCOMMe3c5f6ce1930: [keyserver] new fetcher to get all communities (authored by varun). · Explain WhySep 24 2024, 10:56 AM
This revision was automatically updated to reflect the committed changes.
varun added a commit: rCOMMe3c5f6ce1930: [keyserver] new fetcher to get all communities.

Revision Contents
Changeset List

PathSize
keyserver/
src/
fetchers/
32 lines
lib/
types/
5 lines

Diff 44489

View Options

keyserver/src/fetchers/community-fetchers.js

Loading...
View Options

lib/types/community-types.js

Loading...