Details

Reviewers

• jacek
tomek

Commits

Summary

Move currentTabType state out of the inner class component state, and move it out to the wrapping "Connected" functional component.

As part of the work to refactor ThreadSettingsModal… specifically to turn it into a functional component.

Depends on D3759

Test Plan

Was able to change settings in the ThreadSettingsModal and things continued to look/work as expected.

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

atul published this revision for review.Apr 18 2022, 11:39 AM

tomek added inline comments.

web/modals/threads/thread-settings-modal.react.js
349–352	I know this is pure refactor diff, but just wanted to let you know that there's an inconsistency in what we're storing in state. When we clear the state, we set it to `Object.freeze({})`, but when we update it, we no longer freeze. We should either always freeze or never. I think it should be fine to avoid freezing, as `ThreadChanges` has readonly fields.

This revision is now accepted and ready to land.Apr 19 2022, 3:52 AM

rebase before landing

web/modals/threads/thread-settings-modal.react.js
349–352	Used `Object.freeze()` where it was used initially and ran into flow issues when I removed it: We should either always freeze or never. Updating this diff to "freeze" everywhere.