Page MenuHomePhabricator

[keyserver] Give all members permissions to add users to a community
ClosedPublic

Authored by ashoat on Mar 10 2023, 12:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 7, 9:20 AM
Unknown Object (File)
Tue, Nov 5, 10:06 PM
Unknown Object (File)
Tue, Nov 5, 10:06 PM
Unknown Object (File)
Wed, Oct 30, 3:44 AM
Unknown Object (File)
Tue, Oct 29, 6:24 AM
Unknown Object (File)
Tue, Oct 29, 6:24 AM
Unknown Object (File)
Tue, Oct 29, 6:24 AM
Unknown Object (File)
Tue, Oct 29, 6:21 AM
Subscribers

Details

Summary

Until we have support for roles in the UI, roles are entirely a function of thread type.

This diff updates communities (except GENESIS) so that all members have permissions to add somebody.

It also adds a migration to recalculate permissions for all threads.

Test Plan

I tested this in my local environment:

  1. I ran the migration locally
    • It took about 3.6 minutes, but prod has about 32% more threads than my local environments
    • Prod is also likely slower than my MBP. Assuming it's about 50% slower, I'm estimating it will take about 7.12min for this migration to run on prod
    • For that reason, I'll wait until late in the night before deploying it
  2. I created a new community locally via keyserver/src/scripts/create-community.js
    • I then added a rando test user to it, and confirmed the rando test user could add their friends

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable