Page MenuHomePhabricator

[web-db] restore database Safari support
ClosedPublic

Authored by kamil on Jun 6 2023, 10:21 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 4, 11:30 PM
Unknown Object (File)
Feb 10 2024, 5:52 AM
Unknown Object (File)
Feb 10 2024, 5:52 AM
Unknown Object (File)
Feb 10 2024, 5:50 AM
Unknown Object (File)
Feb 10 2024, 5:32 AM
Unknown Object (File)
Feb 4 2024, 3:57 AM
Unknown Object (File)
Feb 1 2024, 4:59 PM
Unknown Object (File)
Jan 5 2024, 4:04 AM
Subscribers
Tokens
"Party Time" token, awarded by atul.

Details

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 held this revision as a draft.
kamil published this revision for review.Jun 7 2023, 1:52 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
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

This revision was automatically updated to reflect the committed changes.