[lib] Refactor logout in thread activity reducer
ClosedPublic
Actions

Authored by inka on Jan 17 2024, 6:57 AM.

Details

Reviewers

michal
kamil
ginsu

Commits

rCOMM5b7df280c482: [lib] Refactor logout in thread activity reducer

Summary

issue: ENG-5827
We want to remove only the threads related to the keyservers the user is disconnecting from

Test Plan

Ran the provided test

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

inka created this revision.Jan 17 2024, 6:57 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJan 17 2024, 6:57 AM

We are discussing in ENG-6424 if logout should be refactored, but delete account has to be refactored anyway

Harbormaster completed remote builds in B25834: Diff 35723.Jan 17 2024, 7:15 AM

inka requested review of this revision.Jan 17 2024, 7:15 AM

We decided to have a single action for logging out of keyservers and identity.

Undo changes to logout

Harbormaster completed remote builds in B25885: Diff 35780.Jan 18 2024, 3:10 AM

Merge identity delete account with keyserver delete account

Harbormaster completed remote builds in B26010: Diff 35943.Jan 22 2024, 4:35 AM

michal accepted this revision.Jan 29 2024, 7:23 AM

This revision is now accepted and ready to land.Jan 29 2024, 7:23 AM

inka added a child revision: D10778: [web] Handle invalid session downgrade for identity delete account.Jan 29 2024, 7:53 AM

Closed by commit rCOMM5b7df280c482: [lib] Refactor logout in thread activity reducer (authored by inka). · Explain WhyJan 29 2024, 8:42 AM

This revision was automatically updated to reflect the committed changes.

inka added a commit: rCOMM5b7df280c482: [lib] Refactor logout in thread activity reducer.

Revision Contents
Changeset List

Path

Size

lib/

reducers/

thread-activity-reducer.js

17 lines

thread-activity-reducer.test.js

42 lines

Diff 35943

View Options

lib/reducers/thread-activity-reducer.js