Add a comment to describe why we're not using the property we should be

pull/21833/head
Travis Ralston 2018-12-19 10:21:43 -07:00
parent 04c9fff6ce
commit 45f05092ed
1 changed files with 3 additions and 0 deletions

View File

@ -68,6 +68,9 @@ export default class MemberStatusMessageAvatar extends React.Component {
if (ev.getStateKey() !== MatrixClientPeg.get().getUserId()) return; if (ev.getStateKey() !== MatrixClientPeg.get().getUserId()) return;
if (ev.getType() !== "im.vector.user_status") return; if (ev.getType() !== "im.vector.user_status") return;
// TODO: We should be relying on `this.props.member.user._unstable_statusMessage` // TODO: We should be relying on `this.props.member.user._unstable_statusMessage`
// We don't currently because the js-sdk doesn't emit a specific event for this
// change, and we don't want to race it. This should be improved when we rip out
// the im.vector.user_status stuff and replace it with a complete solution.
this.setState({message: ev.getContent()["status"]}); this.setState({message: ev.getContent()["status"]});
}; };