HomePhabricator
Diffusion Comm ee7f8e2a263e

[keyserver] `filterThreadEditAvatarPermission` based on `codeVersion` check

Description

[keyserver] filterThreadEditAvatarPermission based on codeVersion check

Summary: We expect 213 to be the first native client with support for thread avatars. This diff will make sure EDIT_THREAD_AVATAR permissions are INCLUDED for clients >= 213 so the feature works properly.

Test Plan:
Will make sure client threadInfos and keyserver threadInfos match up.

  1. Install old prod build (208?)
  2. Connect to "new" keyserver (ie keyserver after this diff is landed)
  3. Update to 213+ (which includes persist migration)
  4. Connect to "new keyserver and monitor state check mechanism

We expect that the migration will bring the client Redux state "up-to-date" (ie including EDIT_THREAD_AVATAR) and "new keyserver" will include EDIT_THREAD_AVATAR in threadInfos it computes for client... and they'll be consistent.

Reviewers: ashoat, ginsu

Reviewed By: ashoat

Subscribers: tomek

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

Details