Page MenuHomePhabricator

[web] Remove userAgent from Redux state
ClosedPublic

Authored by ashoat on Dec 21 2022, 8:25 PM.
Tags
None
Referenced Files
F2018358: D5985.diff
Sat, Jun 15, 9:14 PM
Unknown Object (File)
Wed, Jun 12, 4:42 PM
Unknown Object (File)
Wed, Jun 12, 4:00 AM
Unknown Object (File)
Tue, Jun 11, 9:34 AM
Unknown Object (File)
Mon, Jun 10, 4:54 PM
Unknown Object (File)
Wed, Jun 5, 3:08 AM
Unknown Object (File)
Wed, Jun 5, 3:08 AM
Unknown Object (File)
Wed, Jun 5, 3:08 AM
Subscribers
None

Details

Summary

Same as the previous diff, basically. We should've had this in a React context... basically wanted server code to know what the user agent was while doing SSR. Now we don't do SSR, and we can just call detectBrowser() without needing to pass anything in.

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Dec 21 2022, 8:35 PM
Harbormaster failed remote builds in B14655: Diff 19991!
tomek added inline comments.
web/chat/chat-message-list.react.js
280–287 ↗(On Diff #19999)

Can we compute this outside the component and avoid using memo?

This revision is now accepted and ready to land.Dec 22 2022, 8:29 AM

No point using props

web/chat/chat-message-list.react.js
280–287 ↗(On Diff #19999)

Good call

This revision was automatically updated to reflect the committed changes.