cleanup

Diebbo · Dec 11, 2024 · ef2d920 · ef2d920
1 parent da1f474
commit ef2d920
Show file tree

Hide file tree

Showing 6 changed files with 24 additions and 131 deletions.
diff --git a/client/components/maps/MapComponent.tsx b/client/components/maps/MapComponent.tsx
@@ -7,10 +7,9 @@ import {
   Popup,
   useMap,
   useMapEvents,
-  CircleMarker,
 } from "react-leaflet";
 import "leaflet/dist/leaflet.css";
-import "leaflet-defaulticon-compatibility/dist/leaflet-defaulticon-compatibility.webpack.css"; // Re-uses images from ~leaflet package
+import "leaflet-defaulticon-compatibility/dist/leaflet-defaulticon-compatibility.webpack.css";
 import * as L from "leaflet";
 import "leaflet-defaulticon-compatibility";
 
@@ -45,16 +44,20 @@ const LocateControl = ({
       toast(
         <div className="bg-white dark:bg-gray-900 border border-gray-200 dark:border-gray-700 rounded-lg shadow-lg">
           <div className="p-4">
-            <h4 className="font-semibold text-gray-900 dark:text-white">{"📍 No permission"}</h4>
-            <p className="text-sm text-red-400 dark:text-red-600">{"You have to allow GPS permission in your browser"}</p>
+            <h4 className="font-semibold text-gray-900 dark:text-white">
+              {"📍 No permission"}
+            </h4>
+            <p className="text-sm text-red-400 dark:text-red-600">
+              {"You have to allow GPS permission in your browser"}
+            </p>
           </div>
         </div>,
         {
-          duration: 5000  ,
-          position: 'top-right',
+          duration: 5000,
+          position: "top-right",
         }
       );
-    }
+    },
   });
   return (
     <div className="leaflet-top leaflet-right">
@@ -64,7 +67,7 @@ const LocateControl = ({
           title="Centra sulla mia posizione"
           onClick={(e) => {
             e.preventDefault();
-            
+
             map.locate();
           }}
           className="inline-flex items-center justify-center w-10 h-10 bg-white text-black"
@@ -153,7 +156,9 @@ const MapComponent: React.FC<MapComponentProps> = ({
                          active:scale-95"
                 onClick={() => {
                   navigator.clipboard.writeText(
-                    `${userLocation[0].toFixed(6)}, ${userLocation[1].toFixed(6)}`,
+                    `${userLocation[0].toFixed(6)}, ${userLocation[1].toFixed(
+                      6
+                    )}`
                   );
                 }}
               >
@@ -243,9 +248,6 @@ const MapComponent: React.FC<MapComponentProps> = ({
                 <h3 className="font-bold text-lg text-blue-900 dark:text-blue-100">
                   {friend.name}
                 </h3>
-                <div className="text-sm text-gray-600 dark:text-gray-300 mt-1">
-                  Ultimo aggiornamento: {/* ADD last GPS update in db */}
-                </div>
                 <button
                   className="mt-3 px-4 py-1.5 bg-blue-600 hover:bg-blue-800
                            text-white rounded-full text-sm font-medium

diff --git a/client/components/navbar/notifications-dropdown.tsx b/client/components/navbar/notifications-dropdown.tsx
@@ -43,7 +43,7 @@ export const NotificationsDropdown = () => {
       setNotifications(data);
     } catch (err) {
       setError(
-        err instanceof Error ? err.message : "Error fetching notifications",
+        err instanceof Error ? err.message : "Error fetching notifications"
       );
     } finally {
       setIsLoading(false);
@@ -52,17 +52,8 @@ export const NotificationsDropdown = () => {
 
   const fetchID = async () => {
     try {
-      /*const response = await fetch("/api/users/id", {
-        method: "GET",
-        credentials: "include",
-      });
-
-      if (!response.ok) {
-        throw new Error("Failed to fetch user ID");
-      }*/
-
       const user = await getUser();
-      return user; // Ritorna l'utente
+      return user;
     } catch (err) {
       setError(err instanceof Error ? err.message : "Error fetching user ID");
       throw err; // Rilancia l'errore per gestirlo in initializeSocket
@@ -113,9 +104,9 @@ export const NotificationsDropdown = () => {
               {
                 duration: 5000,
                 position: "top-right",
-              },
+              }
             );
-          },
+          }
         );
 
         // Gestione errori di connessione
@@ -126,15 +117,14 @@ export const NotificationsDropdown = () => {
       } catch (error) {
         console.error("Error initializing socket:", error);
         setError(
-          error instanceof Error ? error.message : "Error initializing socket",
+          error instanceof Error ? error.message : "Error initializing socket"
         );
       } finally {
         setIsLoading(false);
       }
     };
 
     // Fetch le notifiche iniziali
-    // Avvia l'inizializzazione
     initializeSocket();
 
     // Cleanup
@@ -162,7 +152,7 @@ export const NotificationsDropdown = () => {
       setNotifications([]); // Aggiorna lo stato locale dopo la cancellazione
     } catch (err) {
       setError(
-        err instanceof Error ? err.message : "Error clearing notifications",
+        err instanceof Error ? err.message : "Error clearing notifications"
       );
     }
   };
@@ -179,11 +169,11 @@ export const NotificationsDropdown = () => {
       }
 
       setNotifications((prev) =>
-        prev.filter((n) => n._id !== notification._id),
+        prev.filter((n) => n._id !== notification._id)
       );
     } catch (err) {
       setError(
-        err instanceof Error ? err.message : "Error deleting notification",
+        err instanceof Error ? err.message : "Error deleting notification"
       );
     }
   };
@@ -258,6 +248,7 @@ export const NotificationsDropdown = () => {
                 <DropdownItem
                   key={index}
                   className="px-2 py-2"
+                  textValue={notification.title}
                   onClick={(e) => {
                     e.preventDefault();
                     window.location.href = notification.link;

diff --git a/client/public/service-worker.js b/client/public/service-worker.js
@@ -7,8 +7,6 @@ self.addEventListener("push", function (event) {
 
     const notificationPromise = self.registration.showNotification(data.title, {
       body: data.body,
-      //icon: "/icon.png",
-      //badge: "/badge.png",
       data: { url: data.data.url },
       actions: [
         {

diff --git a/server/src/notifications.js b/server/src/notifications.js
diff --git a/server/src/routes/time.js b/server/src/routes/time.js
@@ -5,13 +5,6 @@ function createTimeRouter(db) {
   const router = express.Router();
 
   router.post("/time", cookieJwtAuth, function (req, res, next) {
-    // TODO: Remove this
-    /*if (req.user.role !== "admin") {
-      return res
-        .status(403)
-        .json({ message: "You are not authorized to change the time" });
-        }*/
-
     if (!req.body.date) {
       return res.status(400).json({ message: "Time required" });
     }
@@ -28,13 +21,6 @@ function createTimeRouter(db) {
   });
 
   router.delete("/time", cookieJwtAuth, function (req, res, next) {
-    // TODO: Remove this
-    /*if (req.user.role !== "admin") {
-      return res
-        .status(403)
-        .json({ message: "You are not authorized to change the time" });
-        }*/
-
     try {
       db.changeDateTime(null, true);
       console.log("Time has been reset to real time");

diff --git a/server/src/socket.js b/server/src/socket.js
@@ -52,13 +52,6 @@ export function createWebSocket(io, database, sendNotification) {
         // Salva il socket ID dell'utente
         notificationSockets.set(userId, socket.id);
 
-        // Invia una notifica di benvenuto
-        /*socket.emit("new_notification", {
-          title: "🔔 Notifiche attive",
-          body: "Riceverai qui le tue notifiche",
-          timestamp: new Date(),
-        });*/
-
         console.log(`User ${userId} joined notification system< successfully`);
       } catch (error) {
         console.error("Error in notification join handler:", error);
@@ -256,8 +249,7 @@ export async function emitNotification(userId, notification) {
   }
 
   try {
-    //console.log("Trying to emit notification to user", userId, notification);
-    // Invia la notifica alla room personale dell'utente
+    // Invia la notifica all'utente
     ioInstance.to(userId).emit("new_notification", notification);
     console.log(`Notification emitted to user ${userId}`);
     return true;