HomePhabricator
Diffusion Comm eca24273bc62

[CommCoreModule] Update thread `REPLACE_OPERATION` to handle `avatar`

Tags
None
Referenced Files
F471907: caac41.png
Apr 8 2023, 3:38 PM
File Not Attached
F471905: 3afc84.png
Apr 8 2023, 3:38 PM
File Not Attached
F471906: 27604a.png
Apr 8 2023, 3:38 PM
File Not Attached
Subscribers
None

Description

[CommCoreModule] Update thread REPLACE_OPERATION to handle avatar

Summary:
We need to update construction of REPLACE_OPERATION to handle avatar.

After this diff avatar will be persisted in the SQLite threads table and will REHYDRATE Redux.

NOTE: I am NOT going to include a migration to handle avatars that may already be in the Redux store because there should be no avatars on prod as of this diff landing.

Test Plan:

  1. Set thread avatar from native.
  1. Observe that thread avatar is set in serverDB:

27604a.png (122×1 px, 24 KB)

  1. Observe that Redux state on native is updated with avatar:

3afc84.png (182×374 px, 21 KB)

  1. Observe that avatar field is persisted in clientDB threads table.
  1. Observe that avatar is included in SET_CLIENT_DB_STORE action:

caac41.png (1×2 px, 389 KB)

Reviewers: ashoat, ginsu

Reviewed By: ashoat

Subscribers: rohan, tomek

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

Details

Provenance
atulAuthored on Apr 8 2023, 3:35 PM
Reviewer
ashoat
Differential Revision
D7352: [CommCoreModule] Update thread `REPLACE_OPERATION` to handle `avatar`
Parents
rCOMMdc8a7490d7ba: Revert "[native] Temporary changes for staff release"
Branches
Unknown
Tags
Unknown