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
No Lint Coverage - Unit
No Test Coverage
Event Timeline
| lib/shared/chat-message-item-utils.js | ||
|---|---|---|
| 19–27 | Do we need to specify messageInfos here? | |
| lib/shared/chat-message-item-utils.js | ||
|---|---|---|
| 19–27 | 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) { ... } | |