@marcos_feitoza/personal-finance-backen-trades-assets 1.0.4 → 1.1.0

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [1.1.0](https://github.com/MarcosOps/personal-finance-backend-trades-assets/compare/v1.0.4...v1.1.0) (2025-11-25)
+
+
+### Features
+
+* add Schema for updating a Trade ([315b157](https://github.com/MarcosOps/personal-finance-backend-trades-assets/commit/315b157d5de5d5db82570fc941fb117a80f4cbfe))
+
 ## [1.0.4](https://github.com/MarcosOps/personal-finance-backend-trades-assets/compare/v1.0.3...v1.0.4) (2025-11-21)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@marcos_feitoza/personal-finance-backen-trades-assets",
-    "version": "1.0.4",
+    "version": "1.1.0",
     "publishConfig": {
         "access": "public"
     },

package/personal_finance_trades_assets/routers/trades.py

@@ -1,5 +1,6 @@
 from fastapi import APIRouter, Depends, HTTPException
 from sqlalchemy.orm import Session
+from datetime import datetime
 from personal_finance_shared import models, schemas, crud
 from personal_finance_shared.database import get_db
 import logging
@@ -108,10 +109,37 @@ def read_trades(
     current_user: models.User = Depends(get_current_user)
 ):
     logger.info(f"Reading trades for user {current_user.id}, account: {investment_account}")
-    query = db.query(models.Trade).filter(models.Trade.user_id == current_user.id)
+    query = db.query(models.Trade).filter(
+        models.Trade.user_id == current_user.id,
+        models.Trade.deleted_at.is_(None) # Filter out soft-deleted trades
+    )
     if investment_account:
         query = query.filter(models.Trade.investment_account == investment_account)
     
     trades = query.order_by(models.Trade.date.desc()).offset(skip).limit(limit).all()
     logger.info(f"Found {len(trades)} trades for user {current_user.id}")
-    return trades
+    return trades
+
+@router.delete("/{trade_id}", response_model=schemas.TradeResponse)
+def delete_trade(
+    trade_id: int,
+    db: Session = Depends(get_db),
+    current_user: models.User = Depends(get_current_user)
+):
+    logger.info(f"Attempting soft-delete for trade ID: {trade_id} by user: {current_user.id}")
+    db_trade = db.query(models.Trade).filter(
+        models.Trade.id == trade_id,
+        models.Trade.user_id == current_user.id,
+        models.Trade.deleted_at.is_(None) # Ensure it's not already deleted
+    ).first()
+
+    if not db_trade:
+        logger.warning(f"Trade ID: {trade_id} not found or already deleted for user: {current_user.id}")
+        raise HTTPException(status_code=404, detail="Trade not found or already deleted")
+
+    db_trade.deleted_at = datetime.utcnow() # Set deletion timestamp
+    db.add(db_trade)
+    db.commit()
+    db.refresh(db_trade)
+    logger.info(f"Successfully soft-deleted trade ID: {trade_id} for user: {current_user.id}")
+    return db_trade