[web-db] restore database Safari support
ClosedPublic
Actions

Authored by kamil on Jun 6 2023, 10:21 AM.

Details

Reviewers

michal
tomek

Commits

rCOMM21e3077f7193: [web-db] restore database Safari support

Summary

Safari bugs are fixed in the rest of the stack.
(context)

Depends on D8124

Test Plan

Test everything end-to-end on Safari (clearing content on logout, init on login, draft persistence, etc.)

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Jun 6 2023, 10:21 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptJun 6 2023, 10:21 AM

Harbormaster completed remote builds in B20062: Diff 27492.Jun 6 2023, 10:42 AM

kamil published this revision for review.Jun 7 2023, 1:52 AM

tomek accepted this revision.Jun 13 2023, 4:33 AM

tomek added inline comments.

web/database/utils/constants.js
25 ↗	(On Diff #27492)	Does it include mobile Safari?

This revision is now accepted and ready to land.Jun 13 2023, 4:33 AM

kamil added inline comments.Jun 14 2023, 9:20 AM

web/database/utils/constants.js
25 ↗	(On Diff #27492)	If you ask about this variable `DB_SUPPORTED_BROWSERS` - yes. If you ask about entire logic - no, we always check also for supported OS and exclude mobiles

Closed by commit rCOMM21e3077f7193: [web-db] restore database Safari support (authored by kamil). · Explain WhyJun 14 2023, 10:14 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM21e3077f7193: [web-db] restore database Safari support.

atul awarded a token.Jun 19 2023, 3:23 PM

Revision Contents
Changeset List

Path

Size

web/

database/

utils/

constants.js

1 line

Diff 27766

View Options

web/database/utils/constants.js

[web-db] restore database Safari supportClosedPublicActions