Skip to content

Commit aba5946

Browse files
committed
Enhance logging in item update function to improve tracking of condition changes and acquisition dates
- Updated logging parameters in the update function of item.py to use 'note' instead of 'notes' for consistency. - Added old and new condition fields to the logs for better clarity on changes. - Changed logged_at to event_date to standardize date handling in logs.
1 parent 0fc05f9 commit aba5946

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

app/api/item.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,8 +152,10 @@ def update(payload: ItemUpdate, user: User = Depends(authenticate)):
152152
item_id=item.id,
153153
user_id=user.id,
154154
event_type="condition_change",
155-
notes=f"Condition changed from {old_condition or 'unset'} to {payload.condition}",
156-
logged_at=datetime.datetime.utcnow(),
155+
note=f"Condition changed from {old_condition or 'unset'} to {payload.condition}",
156+
old_condition=old_condition,
157+
new_condition=payload.condition,
158+
event_date=datetime.date.today(),
157159
)
158160
db.session.add(log)
159161

@@ -162,8 +164,8 @@ def update(payload: ItemUpdate, user: User = Depends(authenticate)):
162164
item_id=item.id,
163165
user_id=user.id,
164166
event_type="acquired",
165-
notes=f"Acquired on {payload.acquired_date}",
166-
logged_at=datetime.datetime.utcnow(),
167+
note=f"Acquired on {payload.acquired_date}",
168+
event_date=datetime.date.today(),
167169
)
168170
db.session.add(log)
169171

0 commit comments

Comments
 (0)