Page MenuHomePhabricator

[web] Remove timeZone from Redux state
ClosedPublic

Authored by ashoat on Dec 21 2022, 8:24 PM.
Tags
None
Referenced Files
F3523272: D5984.diff
Mon, Dec 23, 8:51 AM
Unknown Object (File)
Wed, Dec 4, 12:27 PM
Unknown Object (File)
Wed, Dec 4, 12:26 PM
Unknown Object (File)
Wed, Dec 4, 12:26 PM
Unknown Object (File)
Wed, Dec 4, 12:25 PM
Unknown Object (File)
Wed, Dec 4, 12:25 PM
Unknown Object (File)
Wed, Dec 4, 12:06 PM
Unknown Object (File)
Nov 7 2024, 2:09 AM
Subscribers
None

Details

Summary

We probably should've never had this in Redux state... would've been more appropriate in React context.

Basically we need to render timestamps in client time instead of server time during SSR, we needed some context to tell components if they needed to shift timestamps to accomodate a timezone.

Following the parent diff, we no longer need to render client timestamps on the server, so we can remove this code.

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

This revision is now accepted and ready to land.Dec 22 2022, 8:28 AM
This revision was automatically updated to reflect the committed changes.