Page MenuHomePhabricator

[lib] add operation for removing all reports on login/logout
ClosedPublic

Authored by kamil on May 22 2023, 10:57 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 4, 12:38 AM
Unknown Object (File)
Feb 19 2024, 9:10 PM
Unknown Object (File)
Feb 19 2024, 8:50 PM
Unknown Object (File)
Feb 19 2024, 2:10 PM
Unknown Object (File)
Feb 19 2024, 1:31 PM
Unknown Object (File)
Feb 19 2024, 12:00 PM
Unknown Object (File)
Feb 19 2024, 11:54 AM
Unknown Object (File)
Feb 3 2024, 6:47 AM
Subscribers

Details

Summary

In this case this change is very simple - so generating only db operations without calling processReportStoreOperations

Depends on D7921

Test Plan
  1. Logout/login user (create some reports previously)
  2. tests

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.May 23 2023, 1:52 AM
tomek added inline comments.
lib/reducers/report-store-reducer.js
83

Are you sure we should remove these on login?

This revision is now accepted and ready to land.May 24 2023, 2:10 AM

rebase

lib/reducers/report-store-reducer.js
83

yes, operations should maintain previous logic, and on the login queuedReports was returned as an empty array