Added 'hasBeenEdited' variable to track if the message was edited. Will be used to displaying 'edited' label.
Details
Details
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Branch
- kuba/messageediting
- Lint
No Lint Coverage - Unit
No Test Coverage
Event Timeline
lib/selectors/chat-selectors.js | ||
---|---|---|
332 ↗ | (On Diff #23479) | |
403–404 ↗ | (On Diff #23479) | To work with a read-only type, you can do this |
lib/types/messages/text.js | ||
32 ↗ | (On Diff #23479) |
|
Comment Actions
Can we avoid putting this in TextMessageInfo? I don't think it belongs there... and you're doing something pretty inconsistent with how we approached reactions.
Why didn't you put this in ChatMessageInfoItem? (In the future, when you are doing something different like this, please expect that your reviewers will call you out on it, and explain it ahead of time.)
Comment Actions
You are right. It should be in ChatMessageInfoItem. Going to update the diff.
I wasn't sure if we want to add it to RobotextChatMessageInfoItem, but reactions are there as well, so I also added it to it.