Differential D14724 Authored by kamil on May 20 2025, 2:01 PM.
Tags None Referenced Files
Subscribers
Details Summary Test Plan Build app, test log out
Diff Detail
Event Timelinekamil held this revision as a draft. Herald added a subscriber: ashoat. · View Herald TranscriptMay 20 2025, 2:01 PM2025-05-20 14:01:23 (UTC+0) kamil edited the summary of this revision. (Show Details)May 20 2025, 2:38 PM2025-05-20 14:38:22 (UTC+0) kamil added inline comments.
Harbormaster completed remote builds in B34206: Diff 48141.May 20 2025, 2:55 PM2025-05-20 14:55:35 (UTC+0) Comment Actions Generally, what do you think about renaming clearSensitiveData() to indicate it also creates a fresh, clean DB? E.g. something like clearDataAndRecreateDB()? This revision is now accepted and ready to land.May 22 2025, 7:54 AM2025-05-22 07:54:52 (UTC+0) Comment Actions
Makes sense, created ENG-10792 to address that Closed by commit rCOMM3029cd8ef90c: [SQLite] move `clearSensitiveData` to `DatabaseManager`. · Explain WhyMay 26 2025, 12:11 PM2025-05-26 12:11:15 (UTC+0) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 48242 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||