HomePhabricator
Diffusion Comm 82470964d9c9

[web] Flip the switch to strip member permissions from `web`

Description

[web] Flip the switch to strip member permissions from web

Summary: This diff increments Redux version and changes web codeVersion check for keyserver hasMinCodeVersion check to NEXT_CODE_VERSION.

Test Plan:
Will go through Test Plan a couple times in my environment.

  1. Patch in this diff
  2. Rebase and "stop on" (edit) "DUMMY` commit
  3. Run yarn dev everywhere and observe the ThreadInfos via Redux DevTools. Observe that there are member permissions for thin threads.
  4. git rebase --continue
  5. Observe the logs in browser dev tools and confirm that migration ran successfully
  6. Observe that thin threads no longer have member permissions via Redux DevTools.

Before (thin, with member permissions):

Screenshot 2024-10-08 at 4.24.03 PM.png (2×2 px, 581 KB)

Before (thick, with member permissions):

Screenshot 2024-10-08 at 4.24.24 PM.png (2×2 px, 598 KB)


After (thin, WITHOUT member permissions):

Screenshot 2024-10-08 at 4.25.16 PM.png (2×2 px, 648 KB)

After (thick, still WITH member permissions):

Screenshot 2024-10-08 at 4.25.38 PM.png (2×2 px, 639 KB)

Reviewers: ashoat

Reviewed By: ashoat

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D13561

Details

Provenance
atulAuthored on Oct 8 2024, 12:22 PM
Reviewer
ashoat
Differential Revision
D13561: [web] Flip the switch to strip member permissions from `web`
Parents
rCOMM1aad44e6fb68: [web] Introduce `web` migration to strip member permissions
Branches
Unknown
Tags
Unknown