[Release] Show verification status in the room summary card - #5196
Conversation
turt2live
added
the
X-Release-Blocker
9239e0f to
14f6d9e
Compare
|
I don't know what the build system is smoking - the |
|
.. or not. The enum doesn't exist on the release branch. |
|
|
||
| const isRoomEncrypted = useIsEncrypted(cli, room); | ||
| const roomContext = useContext(RoomContext); | ||
| const e2eStatus = roomContext['e2eStatus']; |
There was a problem hiding this comment.
| const e2eStatus = roomContext['e2eStatus']; | |
| const e2eStatus = roomContext.e2eStatus; |
There was a problem hiding this comment.
We don't have types for this, so it will fail the linter.
There was a problem hiding this comment.
(for context, because RoomContext has been TSified on develop but not in the RC)
| }; | ||
|
|
||
| const isRoomEncrypted = useIsEncrypted(cli, room); | ||
| const roomContext = useContext(RoomContext); |
There was a problem hiding this comment.
I guess I am missing how the E2E status gets into this context, but presumably you have tested it...
There was a problem hiding this comment.
I have tested it. No idea how it works, and no idea how the code I copy/pasted works.
There was a problem hiding this comment.
I guess I am missing how the E2E status gets into this context, but presumably you have tested it...
The context contains the full RoomView state (IState)
See RoomView doing <RoomContext.Provider value={this.state}>
3 participants
#5195 to release - see #5195 for details.