added ClientRepo class skeleton and pymongo dependency for MongoDB integration #135

Merged
duarteneves5 merged 3 commits from db-interface-clients into main 2025-11-09 19:06:26 +00:00
duarteneves5 commented 2025-11-08 15:01:56 +00:00 (Migrated from github.com)

followed the same model of VehicleRepo

followed the same model of VehicleRepo
vascodias23 (Migrated from github.com) approved these changes 2025-11-09 17:01:28 +00:00
vascodias23 (Migrated from github.com) left a comment

Looks good!

@GFMpt13 please read my other comment and see for yourself if it makes sense.

Looks good! @GFMpt13 please read my other comment and see for yourself if it makes sense.
@ -0,0 +83,4 @@
Note:
Consider implementing soft delete or checking for active work orders
before allowing deletion (business rule validation)
vascodias23 (Migrated from github.com) commented 2025-11-09 16:58:52 +00:00

Good observation! I would recommend listing this feature on the issues so it is not forgotten and someone implements this logic.

Good observation! I would recommend listing this feature on the issues so it is not forgotten and someone implements this logic.
GFMpt13 (Migrated from github.com) reviewed 2025-11-09 18:52:18 +00:00
@ -0,0 +83,4 @@
Note:
Consider implementing soft delete or checking for active work orders
before allowing deletion (business rule validation)
GFMpt13 (Migrated from github.com) commented 2025-11-09 18:52:18 +00:00

@vascodias23 Yes, I agree. We may discuss the use of a soft delete. But the checking for active work orders must be implemented 100%

@vascodias23 Yes, I agree. We may discuss the use of a soft delete. But the checking for active work orders must be implemented 100%
GFMpt13 (Migrated from github.com) approved these changes 2025-11-09 18:54:25 +00:00
GFMpt13 (Migrated from github.com) left a comment

Everything looks good. Nice note about the soft delete/checking for active work orders.
We must implement the check, and pottentially discuss the soft delete.

Everything looks good. Nice note about the soft delete/checking for active work orders. We must implement the check, and pottentially discuss the soft delete.
duarteneves5 (Migrated from github.com) reviewed 2025-11-09 19:06:17 +00:00
@ -0,0 +83,4 @@
Note:
Consider implementing soft delete or checking for active work orders
before allowing deletion (business rule validation)
duarteneves5 (Migrated from github.com) commented 2025-11-09 19:06:17 +00:00

I already created the issue for that, so it will not be forgotten

I already created the issue for that, so it will not be forgotten
Sign in to join this conversation.
No description provided.