You've already forked matrix-react-sdk
							
							
				mirror of
				https://github.com/matrix-org/matrix-react-sdk.git
				synced 2025-11-03 00:33:22 +03:00 
			
		
		
		
	Show user an error if we fail to create a DM for verification (#9624)
Related: https://github.com/vector-im/element-web/issues/23819 This is still pretty poor, but at least we don't get stuck with a 'verifying...' spinner that is a total failure.
This commit is contained in:
		
				
					committed by
					
						
						GitHub
					
				
			
			
				
	
			
			
			
						parent
						
							7136c23aa7
						
					
				
				
					commit
					baaa9f5dd2
				
			@@ -111,8 +111,21 @@ const EncryptionPanel: React.FC<IProps> = (props: IProps) => {
 | 
				
			|||||||
    const onStartVerification = useCallback(async () => {
 | 
					    const onStartVerification = useCallback(async () => {
 | 
				
			||||||
        setRequesting(true);
 | 
					        setRequesting(true);
 | 
				
			||||||
        const cli = MatrixClientPeg.get();
 | 
					        const cli = MatrixClientPeg.get();
 | 
				
			||||||
 | 
					        let verificationRequest_: VerificationRequest;
 | 
				
			||||||
 | 
					        try {
 | 
				
			||||||
            const roomId = await ensureDMExists(cli, member.userId);
 | 
					            const roomId = await ensureDMExists(cli, member.userId);
 | 
				
			||||||
        const verificationRequest_ = await cli.requestVerificationDM(member.userId, roomId);
 | 
					            verificationRequest_ = await cli.requestVerificationDM(member.userId, roomId);
 | 
				
			||||||
 | 
					        } catch (e) {
 | 
				
			||||||
 | 
					            console.error("Error starting verification", e);
 | 
				
			||||||
 | 
					            setRequesting(false);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					            Modal.createDialog(ErrorDialog, {
 | 
				
			||||||
 | 
					                headerImage: require("../../../../res/img/e2e/warning.svg").default,
 | 
				
			||||||
 | 
					                title: _t("Error starting verification"),
 | 
				
			||||||
 | 
					                description: _t("We were unable to start a chat with the other user."),
 | 
				
			||||||
 | 
					            });
 | 
				
			||||||
 | 
					            return;
 | 
				
			||||||
 | 
					        }
 | 
				
			||||||
        setRequest(verificationRequest_);
 | 
					        setRequest(verificationRequest_);
 | 
				
			||||||
        setPhase(verificationRequest_.phase);
 | 
					        setPhase(verificationRequest_.phase);
 | 
				
			||||||
        // Notify the RightPanelStore about this
 | 
					        // Notify the RightPanelStore about this
 | 
				
			||||||
 
 | 
				
			|||||||
@@ -2152,6 +2152,8 @@
 | 
				
			|||||||
    "The homeserver the user you're verifying is connected to": "The homeserver the user you're verifying is connected to",
 | 
					    "The homeserver the user you're verifying is connected to": "The homeserver the user you're verifying is connected to",
 | 
				
			||||||
    "Yours, or the other users' internet connection": "Yours, or the other users' internet connection",
 | 
					    "Yours, or the other users' internet connection": "Yours, or the other users' internet connection",
 | 
				
			||||||
    "Yours, or the other users' session": "Yours, or the other users' session",
 | 
					    "Yours, or the other users' session": "Yours, or the other users' session",
 | 
				
			||||||
 | 
					    "Error starting verification": "Error starting verification",
 | 
				
			||||||
 | 
					    "We were unable to start a chat with the other user.": "We were unable to start a chat with the other user.",
 | 
				
			||||||
    "Nothing pinned, yet": "Nothing pinned, yet",
 | 
					    "Nothing pinned, yet": "Nothing pinned, yet",
 | 
				
			||||||
    "If you have permissions, open the menu on any message and select <b>Pin</b> to stick them here.": "If you have permissions, open the menu on any message and select <b>Pin</b> to stick them here.",
 | 
					    "If you have permissions, open the menu on any message and select <b>Pin</b> to stick them here.": "If you have permissions, open the menu on any message and select <b>Pin</b> to stick them here.",
 | 
				
			||||||
    "Pinned messages": "Pinned messages",
 | 
					    "Pinned messages": "Pinned messages",
 | 
				
			||||||
 
 | 
				
			|||||||
		Reference in New Issue
	
	Block a user