fix: should use data to handle duration modification (#51)

* fix: move duration parameter into request body

* fix: string with '/' causes 404

* fix: delete empty past record

* feat: remove notification api

* feat: support netlify previews

* fix

* add note

* revert to main

---------

Co-authored-by: Ethan Goh <[email protected]>

Author: Zcraft3793

main.py

@@ -23,7 +23,7 @@
 
 app.add_middleware(
     CORSMiddleware,
-    allow_origins=["http://localhost:5173", "https://v4.zvms.site", "https://v4-netlify.zvms.site/"],
+    allow_origins=["http://localhost:5173", "https://v4.zvms.site", "https://v4-netlify.zvms.site/", "https://deploy-preview-*--zvms.netlify.app/"],
     allow_credentials=True,
     allow_methods=["*"],
     allow_headers=["*"],
@@ -71,9 +71,9 @@ async def home():
         "license": "MIT",
         "source": "https://github.com/zvms/zvms4-backend-python.git",
         "apis": {
-            "user": "/api/user",
-            "activity": "/api/activity",
-            "group": "/api/group",
+            "user": "/api/users",
+            "activity": "/api/activities",
+            "group": "/api/groups",
             "exports": "/api/exports",
             "imports": "/api/imports",
             "logs": "/api/logs"

routers/activities_router.py

@@ -443,15 +443,16 @@ async def read_activity(activity_oid: str, user=Depends(get_current_user)):
     return {"status": "ok", "code": 200, "data": activity}
 
 
+class PutActivityDuration(BaseModel):
+    duration: str
+
+
 @router.put('/{activity_oid}/member/{uid}/duration')
-async def update_activity_member_duration(activity_oid: str, uid: str, duration: float, user=Depends(get_current_user),
+async def update_activity_member_duration(activity_oid: str, uid: str, payload: PutActivityDuration, user=Depends(get_current_user),
                                           log=Depends(inject_log)):
     """
     Update activity member duration
     """
-    log.with_text(
-        f"User {await get_user_name(user['id'])} updated activity member {await get_user_name(uid)}'s duration to {duration} in activity {activity_oid}")
-    await log.insert_log()
 
     activity = await db.zvms.activities.find_one(
         {"_id": validate_object_id(activity_oid)}
@@ -477,7 +478,7 @@ async def update_activity_member_duration(activity_oid: str, uid: str, duration:
                         "in": {
                             "$cond": [
                                 {"$eq": ["$$member._id", uid]},
-                                {"$mergeObjects": ["$$member", {"duration": duration}]},
+                                {"$mergeObjects": ["$$member", {"duration": payload.duration}]},
                                 "$$member"
                             ]
                         }
@@ -492,6 +493,10 @@ async def update_activity_member_duration(activity_oid: str, uid: str, duration:
         pipeline
     )
 
+    log.with_text(
+        f"User {await get_user_name(user['id'])} updated activity member {await get_user_name(uid)}'s duration to {duration} in activity {activity_oid}")
+    await log.insert_log()
+
     return {
         "status": "ok",
         "code": 200,