Page MenuHomePhabricator

[lib] Persist messages after user account is deleted
ClosedPublic

Authored by rohan on Sep 20 2022, 2:03 PM.
Tags
None
Referenced Files
F5058067: D5199.id16934.diff
Wed, Mar 26, 2:48 AM
F5058066: D5199.id16932.diff
Wed, Mar 26, 2:48 AM
F5058065: D5199.id.diff
Wed, Mar 26, 2:48 AM
Unknown Object (File)
Mon, Mar 24, 1:02 AM
Unknown Object (File)
Fri, Mar 21, 7:01 AM
Unknown Object (File)
Feb 17 2025, 8:55 PM
Unknown Object (File)
Feb 17 2025, 8:55 PM
Unknown Object (File)
Feb 17 2025, 8:55 PM

Details

Summary

After a user deletes their account, there is a bug that deletes
their messages from another user's device and removes any trace of
conversation. This diff addresses that by instead persisting the messages,
but replacing the deleted user's account name with anonymous to signify a
deleted account.

Addresses: https://linear.app/comm/issue/ENG-1735/messages-of-a-deleted-account-dont-show-up-in-the-app

Test Plan

Simulated a conversation between two individuals and observed
that, as expected, the deleted user is renamed to anonymous and their
messages persist. In a group chat, the same behavior is expected, where
the user is removed from the group but their messages persist and it is
authored by anonymous.

Video Demo:

Diff Detail

Repository
rCOMM Comm
Branch
ENG-1735
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

rohan requested review of this revision.Sep 20 2022, 2:13 PM
rohan edited the test plan for this revision. (Show Details)
rohan attached a referenced file: F176142: ENG-1735 Demo.mov. (Show Details)
ashoat removed 1 blocking reviewer(s): atul.

Short-circuiting review since this was what we discussed in the Linear issue

This revision is now accepted and ready to land.Sep 20 2022, 2:34 PM