From 0740f9ea5ae93ba2ee3293fb1fea69061686eb5c Mon Sep 17 00:00:00 2001
From: Philip Sargent <philip.sargent@gmail.com>
Date: Wed, 8 Feb 2023 23:37:00 +0000
Subject: [PATCH] More todos

---
 core/models/caves.py    | 5 +++++
 core/models/logbooks.py | 2 ++
 2 files changed, 7 insertions(+)

diff --git a/core/models/caves.py b/core/models/caves.py
index a18a996..1ed1605 100644
--- a/core/models/caves.py
+++ b/core/models/caves.py
@@ -28,6 +28,11 @@ todo = """
   these are not just a single field on the Model. Do we ever need more
   than one slug per cave or entrance? Surely that would break everything??
   
+- Can we rewrite things to eliminate the CaveSlug and EntranceSlug Classes and objects? Surely
+  foreign keys work fine ?!
+  
+- Why do we have CaveAndEntrance objects ? Surely entranceletter belong son the Entrance object?
+  
 - Restore constraint:   unique_together = (("area", "kataster_number"), ("area", "unofficial_number"))
 """
 
diff --git a/core/models/logbooks.py b/core/models/logbooks.py
index eac552a..89abca4 100644
--- a/core/models/logbooks.py
+++ b/core/models/logbooks.py
@@ -11,6 +11,8 @@ from troggle.core.models.troggle import Expedition, TroggleModel
 """
 
 todo = """
+- Can we rewrite things to eliminate the CaveSlug and EntranceSlug Classes and objects? Surely
+  foreign keys work fine ?!
 """