feat(household): allow deleting households

Author: SplinterHead

backend/app/main.py

@@ -34,9 +34,6 @@ def get_db():
  db.close()
 
 
-# --- ONBOARDING ENDPOINTS ---
-
-
 @app.get("/system/status")
 def get_system_status(db: Session = Depends(get_db)):
  """Checks if a household exists to determine if onboarding is needed."""
@@ -46,6 +43,9 @@ def get_system_status(db: Session = Depends(get_db)):
  return {"initialized": False, "household": None}
 
 
+# --- HOUSEHOLD ENDPOINTS ---
+
+
 @app.get("/households/")
 def read_households(db: Session = Depends(get_db)):
  """List all households for the switcher menu."""
@@ -61,6 +61,29 @@ def create_household(name: str = Form(...), db: Session = Depends(get_db)):
  return new_household
 
 
+@app.delete("/households/{household_id}")
+def delete_household(household_id: int, db: Session = Depends(get_db)):
+ household = (
+ db.query(models.Household).filter(models.Household.id == household_id).first()
+    )
+ if not household:
+ raise HTTPException(status_code=404, detail="Household not found")
+ policies = (
+ db.query(models.Policy).filter(models.Policy.household_id == household_id).all()
+    )
+ for policy in policies:
+ if policy.document_path and os.path.exists(policy.document_path):
+ try:
+ os.remove(policy.document_path)
+ except Exception as e:
+ print(f"Error deleting file {policy.document_path}: {e}")
+
+ db.delete(household)
+ db.commit()
+
+ return {"ok": True}
+
+
 # --- ASSET ENDPOINTS ---
 
 

backend/app/models.py

@@ -8,9 +8,8 @@ class Household(Base):
  __tablename__ = "households"
 
  id = Column(Integer, primary_key=True, index=True)
- name = Column(String, index=True) # e.g. "The Smith Family"
+ name = Column(String, index=True)
 
- # Relationship to policies
  policies = relationship("Policy", back_populates="household")
  assets = relationship("Asset", back_populates="household")
 
@@ -19,9 +18,9 @@ class Asset(Base):
  __tablename__ = "assets"
  id = Column(Integer, primary_key=True, index=True)
  household_id = Column(Integer, ForeignKey("households.id"))
- name = Column(String) # e.g. "Dad's Honda"
- type = Column(String) # e.g. "Vehicle"
- details = Column(JSON, default={}) # Stores { make: 'Honda', reg: 'ABC' }
+ name = Column(String)
+ type = Column(String)
+ details = Column(JSON, default={})
 
  household = relationship("Household", back_populates="assets")
  policies = relationship("Policy", back_populates="asset")

frontend/src/App.vue

@@ -59,35 +59,78 @@
         <v-card-text class="pa-4">
           <v-list lines="one" border class="rounded-lg mb-4">
             <v-list-item v-for="h in households" :key="h.id" :title="h.name">
+              <template v-slot:subtitle v-if="currentHousehold?.id === h.id">
+                <span class="text-success font-weight-bold text-caption">Currently Active</span>
+              </template>
+
               <template v-slot:append>
-                <v-chip v-if="currentHousehold?.id === h.id" size="x-small" color="success">
-                  Active
-                </v-chip>
+                <v-btn
+ v-if="currentHousehold?.id !== h.id"
+ size="small"
+ variant="text"
+ color="primary"
+ @click="switchHousehold(h)"
+                  >Switch</v-btn
+                >
+
+                <v-btn
+ icon="mdi-delete"
+ size="small"
+ variant="text"
+ color="grey"
+ class="ml-2"
+ @click="confirmDeleteHousehold(h)"
+                ></v-btn>
               </template>
             </v-list-item>
           </v-list>
+
           <div class="d-flex">
             <v-text-field
  v-model="newHouseholdName"
- label="Name"
+ label="New Household Name"
  variant="outlined"
  density="compact"
  hide-details
  class="mr-2"
- />
+ ></v-text-field>
             <v-btn
  color="primary"
  height="40"
  @click="createHousehold(false)"
  :loading="creatingHousehold"
+              >Add</v-btn
             >
-              Add
-            </v-btn>
           </div>
         </v-card-text>
+        <v-card-actions
+          ><v-spacer></v-spacer
+          ><v-btn variant="text" @click="manageDialog = false">Close</v-btn></v-card-actions
+        >
+      </v-card>
+    </v-dialog>
+
+    <v-dialog v-model="deleteHouseholdDialog" max-width="400px">
+      <v-card>
+        <v-card-title class="text-h5 text-red">Delete Household?</v-card-title>
+        <v-card-text>
+          Are you sure you want to delete <strong>{{ householdToDelete?.name }}</strong
+          >? <br /><br />
+          <v-alert type="warning" variant="tonal" density="compact" border="start">
+            This will permanently delete all
+            <strong>Policies, Assets, and Documents</strong> associated with this household.
+          </v-alert>
+        </v-card-text>
         <v-card-actions>
-          <v-spacer />
-          <v-btn variant="text" @click="manageDialog = false"> Close </v-btn>
+          <v-spacer></v-spacer>
+          <v-btn color="grey" variant="text" @click="deleteHouseholdDialog = false">Cancel</v-btn>
+          <v-btn
+ color="red"
+ variant="flat"
+ @click="executeDeleteHousehold"
+ :loading="deletingHousehold"
+            >Delete Forever</v-btn
+          >
         </v-card-actions>
       </v-card>
     </v-dialog>
@@ -107,8 +150,11 @@ export default {
       creatingHousehold: false,
       currencyCode: 'GBP',
       currentHousehold: null,
+      deleteHouseholdDialog: false,
+      deletingHousehold: false,
       drawer: null,
       households: [],
+      householdToDelete: null,
       initialized: false,
       manageDialog: false,
       newHouseholdName: '',
@@ -153,6 +199,36 @@ export default {
     },
  switchHousehold(household) {
  this.currentHousehold = household
+ this.manageDialog = false
+    },
+ confirmDeleteHousehold(household) {
+ this.householdToDelete = household
+ this.deleteHouseholdDialog = true
+    },
+ async executeDeleteHousehold() {
+ if (!this.householdToDelete) return
+ this.deletingHousehold = true
+ try {
+ await axios.delete(`http://localhost:8000/households/${this.householdToDelete.id}`)
+ this.households = this.households.filter((h) => h.id !== this.householdToDelete.id)
+ if (this.currentHousehold && this.currentHousehold.id === this.householdToDelete.id) {
+ if (this.households.length > 0) {
+ this.switchHousehold(this.households[0])
+          } else {
+ this.currentHousehold = null
+ this.showOnboarding = true
+ this.manageDialog = false
+          }
+        }
+
+ this.deleteHouseholdDialog = false
+ this.householdToDelete = null
+      } catch (e) {
+ console.error('Failed to delete household', e)
+ alert('Could not delete household. Ensure all policies are removed or try again.')
+      } finally {
+ this.deletingHousehold = false
+      }
     },
  async createHousehold(isOnboarding) {
  if (!this.newHouseholdName) return

frontend/src/components/PolicyList.vue

@@ -510,7 +510,6 @@ export default {
  this.$emit('delete', this.policyToDelete.id)
  this.deleteDialog = false
  this.policyToDelete = null
-      } else {
       }
     },
  async uploadDocument(file) {
@@ -519,17 +518,12 @@ export default {
  formData.append('file', file)
 
  try {
- // Call the new backend endpoint
  const res = await axios.put(
  `http://localhost:8000/policies/${this.selectedPolicy.id}/document`,
           formData,
           { headers: { 'Content-Type': 'multipart/form-data' } },
         )
-
- // Update local state immediately to show the PDF
  this.selectedPolicy.document_path = res.data.document_path
-
- // Also update the main list so the icon appears in the table behind the modal
  const index = this.policies.findIndex((p) => p.id === this.selectedPolicy.id)
  if (index !== -1) {
  this.policies[index].document_path = res.data.document_path