Paths

Table of Contentst

Differential D11544

[web] Clear database on non-threadStore ops fail
ClosedPublic
Actions

Authored by michal on Wed, Apr 3, 7:29 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 11, 9:29 PM

	Unknown Object (File)
	Wed, Apr 10, 6:21 PM

	Unknown Object (File)
	Wed, Apr 10, 4:12 PM

	Unknown Object (File)
	Wed, Apr 10, 9:38 AM

	Unknown Object (File)
	Tue, Apr 9, 3:02 PM

	Unknown Object (File)
	Mon, Apr 8, 12:56 PM

	Unknown Object (File)
	Wed, Apr 3, 7:40 AM

	Unknown Object (File)
	Wed, Apr 3, 7:39 AM

View All 9 Files

Subscribers

Details

Reviewers

kamil
marcin

Commits

rCOMM9705de4522f9: [web] Clear database on non-threadStore ops fail

Summary

ENG-7587 : Should we clear the web DB when some operations fail?
Clean the db on other (non-draft, non-report) ops.

Depends on D11539

Test Plan

Add throw new Error() on message store rekey op. Try sending a message. App displayed an alert and reloaded.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

michal created this revision.Wed, Apr 3, 7:29 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptWed, Apr 3, 7:29 AM

michal added a child revision: D11545: [web/native/lib] Use message pruner on web.Wed, Apr 3, 7:40 AM

Harbormaster completed remote builds in B27954: Diff 38734.Wed, Apr 3, 7:44 AM

michal requested review of this revision.Wed, Apr 3, 7:44 AM

michal added a child revision: D11563: [web] Fix user not getting logged out on ops failure.Fri, Apr 5, 2:59 AM

Rebase

Harbormaster completed remote builds in B28015: Diff 38842.Fri, Apr 5, 3:24 AM

kamil accepted this revision.Fri, Apr 5, 3:40 AM

kamil added inline comments.

web/shared-worker/utils/store.js
209–216 ↗	(On Diff #38842)	can this be simplified using `entries()`? I am mostly worried about cases when someone adds something to `storeOperations` but forgets add it here so I will also inverse the condition

This revision is now accepted and ready to land.Fri, Apr 5, 3:40 AM

Use entries instead of specifying each ops separately.

Harbormaster completed remote builds in B28027: Diff 38856.Fri, Apr 5, 6:00 AM

Closed by commit rCOMM9705de4522f9: [web] Clear database on non-threadStore ops fail (authored by Michal Gniadek <michal.gniadek@swmansion.com>). · Explain WhyFri, Apr 5, 9:45 AM

This revision was automatically updated to reflect the committed changes.

Michal Gniadek <michal.gniadek@swmansion.com> added a commit: rCOMM9705de4522f9: [web] Clear database on non-threadStore ops fail.

Revision Contents
Changeset List

Path

Size

web/

shared-worker/

utils/

10 lines

Diff 38869

web/shared-worker/utils/store.js

Loading...