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)
Mon, Dec 23, 4:24 PM
Unknown Object (File)
Sun, Dec 8, 5:16 AM
Unknown Object (File)
Sun, Dec 8, 3:39 AM
Unknown Object (File)
Fri, Dec 6, 6:02 AM
Unknown Object (File)
Fri, Dec 6, 6:02 AM
Unknown Object (File)
Nov 7 2024, 9:20 AM
Unknown Object (File)
Nov 5 2024, 10:06 PM
Unknown Object (File)
Nov 5 2024, 10:06 PM
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