Remove alerts and make meetings only editable for creator

Alexandra Zwinger · Alexandra Zwinger · commit 41ed29c61b13 · 2025-05-13T10:31:03.000+02:00
diff --git a/src/components/CalendarComponent.tsx b/src/components/CalendarComponent.tsx
@@ -26,7 +26,7 @@ export default function CalendarComponent(props: { isDialogOpen: boolean }) {
             const response = await getMeetings(axiosInstance);
             setEvents(response);
         } catch (error) {
-            alert(error);
+            console.error(error);
         }
     };
 
diff --git a/src/components/meeting/MeetingDetails.tsx b/src/components/meeting/MeetingDetails.tsx
@@ -1,11 +1,11 @@
-import React, { useEffect, useState } from 'react';
+import React, {useEffect, useState} from 'react';
 import '../../styles/Modal.css';
-import { CuteButton } from "../CuteButton";
-import { MeetingDto } from "../../dtos/MeetingDto";
-import { getUserIdsForMeeting } from "../../api/UserGroupApi";
+import {CuteButton} from "../CuteButton";
+import {MeetingDto} from "../../dtos/MeetingDto";
+import {getUserIdsForMeeting} from "../../api/UserGroupApi";
 import axiosInstance from "../../AxiosConfig";
-import { getUser } from "../../api/UserApi";
-import { UserDto } from "../../dtos/UserDto";
+import {getUser} from "../../api/UserApi";
+import {UserDto} from "../../dtos/UserDto";
 
 interface ModalProps {
     isOpen: boolean;
@@ -15,11 +15,10 @@ interface ModalProps {
     openChooseMeetingModal: () => void;
 }
 
-const MeetingDetails: React.FC<ModalProps> = ({ isOpen, meeting, onClose, openMeetingForm, openChooseMeetingModal }) => {
+const MeetingDetails: React.FC<ModalProps> = ({isOpen, meeting, onClose, openMeetingForm, openChooseMeetingModal}) => {
     const [userIds, setUserIds] = useState<string[]>([]);
-    const [myUser, setMyUser] = useState<UserDto | null>(null);  
-    const myUserId = myUser ? myUser.uuid : "";  
-       
+    const [myUser, setMyUser] = useState<UserDto | null>(null);
+
     useEffect(() => {
         if (isOpen) {
             getUser(axiosInstance)
@@ -41,14 +40,14 @@ const MeetingDetails: React.FC<ModalProps> = ({ isOpen, meeting, onClose, openMe
     }, [isOpen, meeting]);
 
 
-     const updateMeeting = () => {
+    const updateMeeting = () => {
         onClose();
         if (meeting?.repeatable !== "NEVER") openChooseMeetingModal();
         else openMeetingForm();
     };
 
     if (!isOpen || !meeting || !myUser) return null;
-if (!isOpen || !meeting || !myUser) return null;
+    if (!isOpen || !meeting || !myUser) return null;
 
     return (
         <div className="modal-overlay" onClick={onClose}>
@@ -87,19 +86,19 @@ if (!isOpen || !meeting || !myUser) return null;
                         {userIds.length === 0 && <li>-</li>}
                     </ul>
                 </div>
-
-                <div className="flex flex-row gap-4 mb-4 justify-end mt-auto items-center">
-                    <div>
-                        <CuteButton
-                            onClick={updateMeeting}
-                            text={"Meeting bearbeiten"}
-                            bgColor={"#56A095"}
-                            textColor={"#e6ebfc"}
-                            classname={"md:text-base text-sm"}
-                        />
+                {meeting.creator == myUser.uuid && (
+                    <div className="flex flex-row gap-4 mb-4 justify-end mt-auto items-center">
+                        <div>
+                            <CuteButton
+                                onClick={updateMeeting}
+                                text={"Meeting bearbeiten"}
+                                bgColor={"#56A095"}
+                                textColor={"#e6ebfc"}
+                                classname={"md:text-base text-sm"}
+                            />
+                        </div>
                     </div>
-                   
-                </div>
+                )}
             </div>
         </div>
     );
diff --git a/src/components/yourStudies/UserInfo.tsx b/src/components/yourStudies/UserInfo.tsx
@@ -32,7 +32,7 @@ export default function UserInfo() {
             const response = await getMeetingsOfWeek(axiosInstance);
             setWeeklyMeetings(filterMeetingsForCurrentWeek(response));
         } catch (error) {
-            alert("Fehler beim Abrufen der Meetings: " + error);
+            console.error("Fehler beim Abrufen der Meetings: " + error);
         }
     };
 
diff --git a/src/components/yourStudies/UserSettings.tsx b/src/components/yourStudies/UserSettings.tsx
@@ -21,7 +21,7 @@ export default function UserSettings() {
             const response = await getModules(axiosInstance);
             setAllModules(response);
         } catch (error) {
-            alert("Error fetching user modules:" + error);
+            console.error("Error fetching user modules:" + error);
         }
     }
 
@@ -31,7 +31,7 @@ export default function UserSettings() {
             setUser(response);
             setOwnModules(response.modules);
         } catch (error) {
-            alert("Fehler beim Abrufen der UserDaten: " + error);
+            console.error("Fehler beim Abrufen der UserDaten: " + error);
         }
     }
 
@@ -59,7 +59,7 @@ export default function UserSettings() {
             await createModule(axiosInstance, module);
             fetchAllModules();
         } catch (error) {
-            alert("Error fetching user modules:" + error);
+            console.error("Error fetching user modules:" + error);
         }
     }
 
@@ -69,7 +69,7 @@ export default function UserSettings() {
             fetchUserInfo();
             setEditModule(false);
         } catch (error) {
-            alert("Error fetching user modules:" + error);
+            console.error("Error fetching user modules:" + error);
         }
     }
 
diff --git a/src/form/CreateOrUpdateMeetingForm.tsx b/src/form/CreateOrUpdateMeetingForm.tsx
@@ -49,7 +49,7 @@ export function CreateOrUpdateMeetingForm({open, onClose, meeting, onlyThisMeeti
             const response = await getModules(axiosInstance);
             setModuleNames(response);
         } catch (error) {
-            alert("Error fetching user modules:" + error);
+            console.error("Error fetching user modules:" + error);
         }
     }
 
@@ -157,7 +157,7 @@ export function CreateOrUpdateMeetingForm({open, onClose, meeting, onlyThisMeeti
                 e.preventDefault();
                 const form = e.currentTarget;
                 if (!form.checkValidity()) {
-                    alert("Bitte alle erforderlichen Felder ausfüllen!");
+                    console.error("Bitte alle erforderlichen Felder ausfüllen!");
                     return;
                 }
                 handleSave();
diff --git a/src/form/SearchMeetingForm.tsx b/src/form/SearchMeetingForm.tsx
@@ -32,7 +32,7 @@ export function SearchMeetingForm({open, onClose}: MeetingFormProps) {
             const response = await getModules(axiosInstance);
             setModuleNames(response);
         } catch (error) {
-            //alert("Error fetching user modules:" + error);
+            console.error("Error fetching user modules:" + error);
         }
     }
 
@@ -44,7 +44,7 @@ export function SearchMeetingForm({open, onClose}: MeetingFormProps) {
             const response = await getMeetingsForModule(axiosInstance, module);
             setMeetings(response);
         } catch (error) {
-            alert("Fehler beim Abrufen der Meetings: " + error);
+            console.error("Fehler beim Abrufen der Meetings: " + error);
         }
     };
 
@@ -89,7 +89,7 @@ export function SearchMeetingForm({open, onClose}: MeetingFormProps) {
                 e.preventDefault();
                 const form = e.currentTarget;
                 if (!form.checkValidity()) {
-                    alert("Bitte alle erforderlichen Felder ausfüllen!");
+                    console.error("Bitte alle erforderlichen Felder ausfüllen!");
                     return;
                 }
             }}>

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ export default function CalendarComponent(props: { isDialogOpen: boolean }) {`
`26`	`26`	`const response = await getMeetings(axiosInstance);`
`27`	`27`	`setEvents(response);`
`28`	`28`	`} catch (error) {`
`29`		`- alert(error);`
	`29`	`+ console.error(error);`
`30`	`30`	`}`
`31`	`31`	`};`
`32`	`32`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ export default function UserInfo() {`
`32`	`32`	`const response = await getMeetingsOfWeek(axiosInstance);`
`33`	`33`	`setWeeklyMeetings(filterMeetingsForCurrentWeek(response));`
`34`	`34`	`} catch (error) {`
`35`		`- alert("Fehler beim Abrufen der Meetings: " + error);`
	`35`	`+ console.error("Fehler beim Abrufen der Meetings: " + error);`
`36`	`36`	`}`
`37`	`37`	`};`
`38`	`38`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ export default function UserSettings() {`
`21`	`21`	`const response = await getModules(axiosInstance);`
`22`	`22`	`setAllModules(response);`
`23`	`23`	`} catch (error) {`
`24`		`- alert("Error fetching user modules:" + error);`
	`24`	`+ console.error("Error fetching user modules:" + error);`
`25`	`25`	`}`
`26`	`26`	`}`
`27`	`27`
`@@ -31,7 +31,7 @@ export default function UserSettings() {`
`31`	`31`	`setUser(response);`
`32`	`32`	`setOwnModules(response.modules);`
`33`	`33`	`} catch (error) {`
`34`		`- alert("Fehler beim Abrufen der UserDaten: " + error);`
	`34`	`+ console.error("Fehler beim Abrufen der UserDaten: " + error);`
`35`	`35`	`}`
`36`	`36`	`}`
`37`	`37`
`@@ -59,7 +59,7 @@ export default function UserSettings() {`
`59`	`59`	`await createModule(axiosInstance, module);`
`60`	`60`	`fetchAllModules();`
`61`	`61`	`} catch (error) {`
`62`		`- alert("Error fetching user modules:" + error);`
	`62`	`+ console.error("Error fetching user modules:" + error);`
`63`	`63`	`}`
`64`	`64`	`}`
`65`	`65`
`@@ -69,7 +69,7 @@ export default function UserSettings() {`
`69`	`69`	`fetchUserInfo();`
`70`	`70`	`setEditModule(false);`
`71`	`71`	`} catch (error) {`
`72`		`- alert("Error fetching user modules:" + error);`
	`72`	`+ console.error("Error fetching user modules:" + error);`
`73`	`73`	`}`
`74`	`74`	`}`
`75`	`75`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ export function CreateOrUpdateMeetingForm({open, onClose, meeting, onlyThisMeeti`
`49`	`49`	`const response = await getModules(axiosInstance);`
`50`	`50`	`setModuleNames(response);`
`51`	`51`	`} catch (error) {`
`52`		`- alert("Error fetching user modules:" + error);`
	`52`	`+ console.error("Error fetching user modules:" + error);`
`53`	`53`	`}`
`54`	`54`	`}`
`55`	`55`
`@@ -157,7 +157,7 @@ export function CreateOrUpdateMeetingForm({open, onClose, meeting, onlyThisMeeti`
`157`	`157`	`e.preventDefault();`
`158`	`158`	`const form = e.currentTarget;`
`159`	`159`	`if (!form.checkValidity()) {`
`160`		`- alert("Bitte alle erforderlichen Felder ausfüllen!");`
	`160`	`+ console.error("Bitte alle erforderlichen Felder ausfüllen!");`
`161`	`161`	`return;`
`162`	`162`	`}`
`163`	`163`	`handleSave();`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ export function SearchMeetingForm({open, onClose}: MeetingFormProps) {`
`32`	`32`	`const response = await getModules(axiosInstance);`
`33`	`33`	`setModuleNames(response);`
`34`	`34`	`} catch (error) {`
`35`		`- //alert("Error fetching user modules:" + error);`
	`35`	`+ console.error("Error fetching user modules:" + error);`
`36`	`36`	`}`
`37`	`37`	`}`
`38`	`38`
`@@ -44,7 +44,7 @@ export function SearchMeetingForm({open, onClose}: MeetingFormProps) {`
`44`	`44`	`const response = await getMeetingsForModule(axiosInstance, module);`
`45`	`45`	`setMeetings(response);`
`46`	`46`	`} catch (error) {`
`47`		`- alert("Fehler beim Abrufen der Meetings: " + error);`
	`47`	`+ console.error("Fehler beim Abrufen der Meetings: " + error);`
`48`	`48`	`}`
`49`	`49`	`};`
`50`	`50`
`@@ -89,7 +89,7 @@ export function SearchMeetingForm({open, onClose}: MeetingFormProps) {`
`89`	`89`	`e.preventDefault();`
`90`	`90`	`const form = e.currentTarget;`
`91`	`91`	`if (!form.checkValidity()) {`
`92`		`- alert("Bitte alle erforderlichen Felder ausfüllen!");`
	`92`	`+ console.error("Bitte alle erforderlichen Felder ausfüllen!");`
`93`	`93`	`return;`
`94`	`94`	`}`
`95`	`95`	`}}>`