Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- arcpatch-D13262
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
web/input/input-state-container.react.js | ||
---|---|---|
917–920 | Technically, now upload type should be always encrypted_{photo,video}, so we could remove the else branch and use invariant here |
Comment Actions
It seems like we can delete commStaffCommunity and threadInfoInsideCommunity from the codebase as well. What do you think?
web/input/input-state-container.react.js | ||
---|---|---|
917–920 | I think that would be a good idea. It's better to get rid of this "dead code" and simplify InputStateContainer. Can you make those changes on both native and web before landing? |
web/input/input-state-container.react.js | ||
---|---|---|
917–920 | Sure! I'll do this |
web/input/input-state-container.react.js | ||
---|---|---|
917–920 | Great – just making sure you didn't miss that I requested to delete commStaffCommunity and threadInfoInsideCommunity as well |