This will make it easier to change its behavior in a later diff.
Depends on D13681
Details
Details
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| lib/shared/chat-message-item-utils.js | ||
|---|---|---|
| 19–27 ↗ | (On Diff #45041) | Do we need to specify messageInfos here? |
| lib/shared/chat-message-item-utils.js | ||
|---|---|---|
| 19–27 ↗ | (On Diff #45041) | Yes, it's necessary. Without explicitly listing messageInfos here, Flow deduces that the ... can contain messageInfos. As a result, Flow isn't able to narrow the type when we do a check like this: if (item.messageInfos) { ... } |