galleria tue foto

api.py

@@ -9,6 +9,7 @@ from datetime import datetime
 from logging.handlers import TimedRotatingFileHandler
 from pathlib import Path
 from typing import Any
+from uuid import uuid4
 
 import cloudinary
 import cloudinary.uploader
@@ -391,16 +392,48 @@ def ensure_user_plants_table(conn: sqlite3.Connection) -> None:
         conn.commit()
     except Exception:
         pass  # Column already exists
+
+    conn.execute(
+        """
+        CREATE TABLE IF NOT EXISTS user_plant_photos (
+            id INTEGER PRIMARY KEY AUTOINCREMENT,
+            plant_id INTEGER NOT NULL,
+            photo_url TEXT NOT NULL,
+            created_at TEXT NOT NULL,
+            FOREIGN KEY (plant_id) REFERENCES user_plants(id) ON DELETE CASCADE
+        )
+        """
+    )
+    conn.execute(
+        "CREATE INDEX IF NOT EXISTS idx_user_plant_photos_plant_id ON user_plant_photos(plant_id)"
+    )
     conn.commit()
 
 
-def _user_plant_row_to_payload(row: sqlite3.Row) -> dict[str, Any]:
+def _get_user_plant_photo_urls(conn: sqlite3.Connection, plant_id: int, fallback_url: str | None) -> list[str]:
+    rows = conn.execute(
+        "SELECT photo_url FROM user_plant_photos WHERE plant_id = ? ORDER BY id DESC",
+        (plant_id,),
+    ).fetchall()
+    urls = [str(r["photo_url"] or "").strip() for r in rows if str(r["photo_url"] or "").strip()]
+    if urls:
+        return urls
+
+    fallback = str(fallback_url or "").strip()
+    return [fallback] if fallback else []
+
+
+def _user_plant_row_to_payload(conn: sqlite3.Connection, row: sqlite3.Row) -> dict[str, Any]:
+    plant_id = int(row["id"])
+    fallback_photo = row["user_photo_url"] if "user_photo_url" in row.keys() else None
+    photo_urls = _get_user_plant_photo_urls(conn, plant_id, fallback_photo)
     return {
-        "id": int(row["id"]),
+        "id": plant_id,
         "plant_name": row["plant_name"],
         "user_given_name": row["user_given_name"],
         "user": row["user_email"] or row["user_id"],
-        "user_photo_url": row["user_photo_url"] if "user_photo_url" in row.keys() else None,
+        "user_photo_url": (photo_urls[0] if photo_urls else None),
+        "user_photos": photo_urls,
         "created_at_iso": row["created_at"],
         "created_at": _format_datetime_display(row["created_at"]),
     }
@@ -439,8 +472,7 @@ def create_user_plant(plant_name: str, user_given_name: str, user: dict[str, Any
             ),
             (cursor.lastrowid,),
         ).fetchone()
-
-    return _user_plant_row_to_payload(row)
+        return _user_plant_row_to_payload(conn, row)
 
 
 def list_user_plants(user: dict[str, Any]) -> list[dict[str, Any]]:
@@ -457,8 +489,7 @@ def list_user_plants(user: dict[str, Any]) -> list[dict[str, Any]]:
             ),
             (user_id,),
         ).fetchall()
-
-    return [_user_plant_row_to_payload(row) for row in rows]
+        return [_user_plant_row_to_payload(conn, row) for row in rows]
 
 
 def delete_user_plant_by_id(user: dict[str, Any], plant_id: int) -> bool:
@@ -476,6 +507,11 @@ def delete_user_plant_by_id(user: dict[str, Any], plant_id: int) -> bool:
         if existing is None:
             return False
 
+        conn.execute(
+            "DELETE FROM user_plant_photos WHERE plant_id = ?",
+            (plant_id,),
+        )
+
         conn.execute(
             "DELETE FROM user_plants WHERE id = ? AND user_id = ?",
             (plant_id, user_id),
@@ -513,10 +549,9 @@ def update_user_plant_created_at_by_id(user: dict[str, Any], plant_id: int, crea
             ),
             (plant_id, user_id),
         ).fetchone()
-
-    if row is None:
-        return None
-    return _user_plant_row_to_payload(row)
+        if row is None:
+            return None
+        return _user_plant_row_to_payload(conn, row)
 
 
 def _build_profile_context(profile: dict[str, Any] | None) -> str:
@@ -959,8 +994,8 @@ async def upload_user_plant_photo(
         result = cloudinary.uploader.upload(
             tmp_path,
             folder="green-assistant/user-plants",
-            public_id=f"plant_{plant_id}_user_{user_id[:16]}",
-            overwrite=True,
+            public_id=f"plant_{plant_id}_user_{user_id[:12]}_{uuid4().hex[:10]}",
+            overwrite=False,
             resource_type="image",
             transformation=[{"width": 1200, "crop": "limit", "quality": "auto:good"}],
         )
@@ -974,6 +1009,11 @@ async def upload_user_plant_photo(
     # Save URL to DB
     with get_user_plants_db_connection() as conn:
         ensure_user_plants_table(conn)
+        created_at = datetime.utcnow().replace(microsecond=0).isoformat() + "Z"
+        conn.execute(
+            "INSERT INTO user_plant_photos (plant_id, photo_url, created_at) VALUES (?, ?, ?)",
+            (plant_id, photo_url, created_at),
+        )
         conn.execute(
             "UPDATE user_plants SET user_photo_url = ? WHERE id = ? AND user_id = ?",
             (photo_url, plant_id, user_id),
@@ -984,9 +1024,10 @@ async def upload_user_plant_photo(
             "FROM user_plants WHERE id = ?",
             (plant_id,),
         ).fetchone()
+        updated_payload = _user_plant_row_to_payload(conn, updated_row)
 
     _log_api("/user/plants/{plant_id}/photo", "uploaded", {"plant_id": plant_id})
-    return JSONResponse(content={"updated": _user_plant_row_to_payload(updated_row)})
+    return JSONResponse(content={"updated": updated_payload})
 
 
 @app.get("/health")

data/user_plants.db

@@ -1,3 +1,3 @@
 version https://git-lfs.github.com/spec/v1
-oid sha256:2a08c80a4d4957fb04471f899b1cd29f7475b830ea28f7c03f2a455ed78acc17
-size 16384
+oid sha256:17c8484b3683c813b0fa16716623911dc62f4498e0cf5d73b40f04f6552fbd4b
+size 24576

pwa-app/src/App.jsx

@@ -177,6 +177,19 @@ export default function App({ googleClientIdConfigured = false }) {
   }, [plantCard]);
 
   const activeImage = galleryImages.length ? galleryImages[imageIndex % galleryImages.length] : "";
+  const myPlantUserPhotos = useMemo(() => {
+    if (!selectedMyPlant) {
+      return [];
+    }
+
+    const raw = Array.isArray(selectedMyPlant.user_photos) && selectedMyPlant.user_photos.length
+      ? selectedMyPlant.user_photos
+      : (selectedMyPlant.user_photo_url ? [selectedMyPlant.user_photo_url] : []);
+
+    return raw
+      .map((url) => toAbsoluteImage(String(url || "").trim()))
+      .filter(Boolean);
+  }, [selectedMyPlant]);
 
   const wateringMonthCalendar = useMemo(() => {
     if (!wateringSchedule.length) {
@@ -1144,7 +1157,15 @@ export default function App({ googleClientIdConfigured = false }) {
 
           {!!myPlants.length && (
             <div className="my-plants-list">
-              {myPlants.map((item) => (
+              {myPlants.map((item) => {
+                const cardPhoto = Array.isArray(item.user_photos) && item.user_photos.length
+                  ? item.user_photos[0]
+                  : item.user_photo_url;
+                const photoCount = Array.isArray(item.user_photos)
+                  ? item.user_photos.length
+                  : (item.user_photo_url ? 1 : 0);
+
+                return (
                 <article
                   key={item.id}
                   className={`my-plant-item ${selectedMyPlant?.id === item.id ? "active" : ""}`}
@@ -1169,9 +1190,9 @@ export default function App({ googleClientIdConfigured = false }) {
                       {deletingPlantId === item.id ? "Elimino..." : "Elimina"}
                     </button>
                   </div>
-                  {item.user_photo_url && (
+                  {cardPhoto && (
                     <img
-                      src={item.user_photo_url}
+                      src={toAbsoluteImage(cardPhoto)}
                       alt={`Foto di ${item.user_given_name}`}
                       className="my-plant-item-photo"
                     />
@@ -1184,10 +1205,11 @@ export default function App({ googleClientIdConfigured = false }) {
                     onClick={(event) => { event.stopPropagation(); openPlantPhotoDialog(item.id); }}
                     disabled={uploadingPhotoId === item.id}
                   >
-                    {uploadingPhotoId === item.id ? "Carico..." : (item.user_photo_url ? "📷 Cambia foto" : "📷 Aggiungi foto")}
+                    {uploadingPhotoId === item.id ? "Carico..." : (photoCount > 0 ? `📷 Aggiungi altra foto (${photoCount})` : "📷 Aggiungi foto")}
                   </button>
                 </article>
-              ))}
+                );
+              })}
             </div>
           )}
         </section>
@@ -1204,12 +1226,21 @@ export default function App({ googleClientIdConfigured = false }) {
             {selectedMyPlant?.user_given_name && (
               <p>Il tuo nome: {selectedMyPlant.user_given_name}</p>
             )}
-            {selectedMyPlant?.user_photo_url && (
-              <img
-                src={selectedMyPlant.user_photo_url}
-                alt={`La tua foto di ${selectedMyPlant.user_given_name}`}
-                className="my-plant-detail-photo"
-              />
+            {!!myPlantUserPhotos.length && (
+              <div className="my-plant-user-gallery">
+                <p className="my-plant-user-gallery-title">Le tue foto</p>
+                <div className="my-plant-user-gallery-grid">
+                  {myPlantUserPhotos.map((photoUrl, idx) => (
+                    <img
+                      key={`${photoUrl}-${idx}`}
+                      src={photoUrl}
+                      alt={`Foto ${idx + 1} di ${selectedMyPlant?.user_given_name || "questa pianta"}`}
+                      className="my-plant-detail-photo"
+                      loading="lazy"
+                    />
+                  ))}
+                </div>
+              </div>
             )}
             {myPlantCard.common_name && <p>Nome comune: {myPlantCard.common_name}</p>}
 

pwa-app/src/styles.css

@@ -629,10 +629,25 @@ pre {
 
 .my-plant-detail-photo {
   width: 100%;
-  max-height: 260px;
+  height: 180px;
   object-fit: cover;
   border-radius: 8px;
-  margin: 0.6rem 0;
+}
+
+.my-plant-user-gallery {
+  margin: 0.55rem 0 0.7rem;
+}
+
+.my-plant-user-gallery-title {
+  margin: 0 0 0.35rem;
+  font-weight: 700;
+  color: #285943;
+}
+
+.my-plant-user-gallery-grid {
+  display: grid;
+  grid-template-columns: repeat(2, minmax(0, 1fr));
+  gap: 0.45rem;
 }
 
 .my-plant-item-head {