application-services icon indicating copy to clipboard operation
application-services copied to clipboard

Extend deleteHistory of PlacesHistoryStorage

Open mavduevskiy opened this issue 2 years ago • 0 comments

Relates to jira ticket ..FNXV2-19087

Background: We are adding a new synced history screen in the android app. After coming to the history screen, users will see only local history, having a synced history button. Clicking it will lead to another history screen, with the synced from other devices items. Both screens have a "remove all" button, and we would like them to behave differently. Currently, deleteEverything method does what it says, but we need a way to delete everything that appeared on the synced history screen.

Requirements: PlacesHistoryStorage has a method deleteEverything(). We would like to extend it to accept is_remote parameter, so we could have control over what HistoryVisitInfo would be deleted.

┆Issue is synchronized with this Jira Task ┆epic: MR2 History Metadata Sync support

mavduevskiy avatar May 25 '22 23:05 mavduevskiy