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)
Sun, Mar 10, 9:57 PM
Unknown Object (File)
Fri, Mar 8, 6:46 PM
Unknown Object (File)
Fri, Mar 8, 6:46 PM
Unknown Object (File)
Tue, Mar 5, 11:29 PM
Unknown Object (File)
Tue, Mar 5, 11:08 PM
Unknown Object (File)
Feb 23 2024, 8:45 AM
Unknown Object (File)
Feb 23 2024, 8:15 AM
Unknown Object (File)
Feb 23 2024, 7:44 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