-
-
Notifications
You must be signed in to change notification settings - Fork 832
Add 'send without verifying' to status bar #1695
Conversation
Fixes other part of element-hq/element-web#5936
@@ -24,7 +24,11 @@ import WhoIsTyping from '../../WhoIsTyping'; | |||
import MatrixClientPeg from '../../MatrixClientPeg'; | |||
import MemberAvatar from '../views/avatars/MemberAvatar'; | |||
import Resend from '../../Resend'; | |||
import { showUnknownDeviceDialogForMessages } from '../../cryptodevices'; | |||
import { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should we import * as cryptodevices from '../../cryptodevices'
rather than having a long list?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
@@ -289,10 +300,11 @@ module.exports = React.createClass({ | |||
if (hasUDE) { | |||
title = _t("Message not sent due to unknown devices being present"); | |||
content = _t( | |||
"<showDevicesText>Show devices</showDevicesText> or <cancelText>cancel all</cancelText>.", | |||
"<showDevicesText>Show devices</showDevicesText>, <sendAnywayText>send without verifying</sendAnywayText> or <cancelText>cancel all</cancelText>.", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's not verifying we care about - it's the fact that they are unknown. I worry that this wording adds to the general confusion that you need to have verified devices before e2e will work.
We could try something like "mark devices known and send"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, true - let's try that
tests seem to be failing? |
Tests are failing because Travis is broken while they faff around shifting stuff between container based and non-container based infra or whatever they're doing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, modulo the tests still being sad
Fixes other part of element-hq/element-web#5936