[keyserver] introduce a table for communities
ClosedPublic
Actions

Authored by ginsu on May 6 2024, 6:48 AM.

Details

Reviewers

tomek
kamil

Commits

rCOMMfb5452a17880: [keyserver] introduce a table for communities

Summary

This new table will be used to store information about a community. For the scope of the project, I intentially made the schema for the table very limited + only focused on tagging a farcaster channel, and figured we could always alter the table in the future to include other things like invite links when I have more free cycles

Depends on D11894

Test Plan

Ran migration and confirmed the table was created. Also created a new db and confirmed that this also creates the communites table.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ginsu created this revision.May 6 2024, 6:48 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMay 6 2024, 6:48 AM

ginsu added reviewers: tomek, kamil.May 6 2024, 6:50 AM

Harbormaster completed remote builds in B28698: Diff 39846.May 6 2024, 7:08 AM

ginsu requested review of this revision.May 6 2024, 7:08 AM

ashoat added inline comments.May 6 2024, 11:51 AM

keyserver/src/database/migration-config.js
775–789 ↗	(On Diff #39846)	Please follow our conventions for alligning these

tomek requested changes to this revision.May 7 2024, 2:04 AM

tomek added inline comments.

keyserver/src/database/migration-config.js
775–811 ↗	(On Diff #39846)	We don't need to have two separate queries from selecting and inserting. It is more efficient to have a single query - the data won't go through the JS. Also, simpler code means that it's more maintainable. This would be more important for non-migration code, but I still think we should make this change.

This revision now requires changes to proceed.May 7 2024, 2:04 AM

ginsu added a child revision: D11904: [keyserver] modify thread creator + deleter endpoints to update communities table.May 7 2024, 10:20 AM

address comments

ginsu added inline comments.May 7 2024, 10:41 AM

keyserver/src/database/migration-config.js
775–811 ↗	(On Diff #39846)	Makes sense! Agree this is a lot more simple + easier to understand

Harbormaster completed remote builds in B28742: Diff 39901.May 7 2024, 10:55 AM

tomek accepted this revision.May 8 2024, 2:47 AM

tomek added inline comments.

keyserver/src/database/migration-config.js
786–788 ↗	(On Diff #39901)	Please update the indentation to match our conventions before landing.
keyserver/src/database/setup-db.js
478 ↗	(On Diff #39901)	We have to remember that there should be created a row in the `communities` table for this thread.