fix: remove form param submission bug

wireapp · Nov 9, 2023 · 5e70571 · 5e70571
1 parent 0c84609
commit 5e70571
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/script/auth/component/JoinGuestLinkPasswordModal.tsx b/src/script/auth/component/JoinGuestLinkPasswordModal.tsx
@@ -45,7 +45,8 @@ const JoinGuestLinkPasswordModal: React.FC<JoinGuestLinkPasswordModalProps> = ({
   const {formatMessage: _} = useIntl();
   const [passwordValue, setPasswordValue] = useState<string>('');
 
-  const onSubmit = () => {
+  const onSubmit = (event: React.FormEvent<HTMLFormElement | HTMLButtonElement>) => {
+    event.preventDefault();
     onSubmitPassword(passwordValue);
   };
 
@@ -70,7 +71,7 @@ const JoinGuestLinkPasswordModal: React.FC<JoinGuestLinkPasswordModalProps> = ({
         <Form
           name="guest-password-join-form"
           data-uie-name="guest-password-join-form"
-          onSubmit={onSubmit}
+          onSubmit={(event: React.FormEvent<HTMLFormElement>) => onSubmit(event)}
           autoComplete="off"
         >
           <Input
@@ -98,7 +99,7 @@ const JoinGuestLinkPasswordModal: React.FC<JoinGuestLinkPasswordModalProps> = ({
           block
           type="button"
           disabled={!passwordValue}
-          onClick={onSubmit}
+          onClick={(event: React.FormEvent<HTMLButtonElement>) => onSubmit(event)}
           data-uie-name="guest-link-join-submit-button"
         >
           {_(joinGuestLinkPasswordModalStrings.joinConversation)}

diff --git a/src/script/auth/module/action/ConversationAction.ts b/src/script/auth/module/action/ConversationAction.ts
@@ -46,7 +46,7 @@ export class ConversationAction {
     code: string,
     uri?: string,
     password?: string,
-  ): ThunkAction<Promise<ConversationEvent | undefined>> => {
+  ): ThunkAction<Promise<ConversationEvent>> => {
     return async (dispatch, getState, {apiClient}) => {
       dispatch(ConversationActionCreator.startJoinConversationByCode());
       try {
@@ -66,7 +66,7 @@ export class ConversationAction {
               has_password: true,
             } as ConversationJoinData),
           );
-          return undefined;
+          throw error;
         }
         dispatch(ConversationActionCreator.failedJoinConversationByCode(error));
         throw error;