feat: Enhance Transportation & Lodging with visits, tags, and multi-collection support

This adds significant new functionality to Transportation and Lodging models:

## Visits Support
- Add visits to Transportation and Lodging (like Locations)
- Track visit dates, notes, and user who created each visit
- Visits step in modal for creating/editing

## Tags Support
- Add tags field to Transportation and Lodging
- Display tags on detail pages
- Full-text search by tags

## Multi-Collection Support
- Change collection field from ForeignKey to ManyToMany
- Items can now belong to multiple collections
- Link existing items to collections from collection detail page

## Standalone List Pages
- New /transportations and /lodging pages with filtering
- Filter by type, visited status, public/private
- Search by name and location
- Consistent with /locations page design

## UI Enhancements
- Visited badges on Transportation and Lodging cards
- Dashboard displays Transportation and Lodging
- Navigation links to new pages

Author: wanoo

backend/server/adventures/admin.py

@@ -110,9 +110,29 @@ def image_display(self, obj):
 
 
 class VisitAdmin(admin.ModelAdmin):
-    list_display = ('location', 'start_date', 'end_date', 'notes')
+    list_display = ('get_parent', 'get_parent_type', 'start_date', 'end_date', 'user', 'notes')
     list_filter = ('start_date', 'end_date')
-    search_fields = ('notes',)
+    search_fields = ('notes', 'location__name', 'transportation__name', 'lodging__name')
+
+    def get_parent(self, obj):
+        if obj.location:
+            return obj.location.name
+        elif obj.transportation:
+            return obj.transportation.name
+        elif obj.lodging:
+            return obj.lodging.name
+        return 'Unknown'
+    get_parent.short_description = 'Parent'
+
+    def get_parent_type(self, obj):
+        if obj.location:
+            return 'Location'
+        elif obj.transportation:
+            return 'Transportation'
+        elif obj.lodging:
+            return 'Lodging'
+        return 'Unknown'
+    get_parent_type.short_description = 'Type'
 
 
     def image_display(self, obj):

backend/server/adventures/migrations/0072_visit_user.py

@@ -0,0 +1,21 @@
+# Add user field to Visit model for tracking who created each visit
+
+from django.conf import settings
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('adventures', '0071_alter_collectionitineraryitem_unique_together_and_more'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='visit',
+            name='user',
+            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.CASCADE, related_name='visits', to=settings.AUTH_USER_MODEL),
+        ),
+    ]

backend/server/adventures/migrations/0073_populate_visit_users.py

@@ -0,0 +1,33 @@
+# Data migration to populate user field on existing visits
+
+from django.db import migrations
+
+
+def populate_visit_users(apps, schema_editor):
+    """
+    Assign existing visits to the location's owner.
+    This ensures existing visits have a user for display purposes.
+    """
+    Visit = apps.get_model('adventures', 'Visit')
+
+    for visit in Visit.objects.filter(user__isnull=True).select_related('location'):
+        if visit.location and visit.location.user:
+            visit.user = visit.location.user
+            visit.save(update_fields=['user'])
+
+
+def reverse_populate(apps, schema_editor):
+    """Reverse migration - clear user from visits."""
+    Visit = apps.get_model('adventures', 'Visit')
+    Visit.objects.update(user=None)
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('adventures', '0072_visit_user'),
+    ]
+
+    operations = [
+        migrations.RunPython(populate_visit_users, reverse_populate),
+    ]

backend/server/adventures/migrations/0074_visit_transportation_lodging.py

@@ -0,0 +1,50 @@
+# Generated for adding Transportation and Lodging to Visit model
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('adventures', '0073_populate_visit_users'),
+    ]
+
+    operations = [
+        # Make location nullable since visits can now be for transportation/lodging
+        migrations.AlterField(
+            model_name='visit',
+            name='location',
+            field=models.ForeignKey(
+                blank=True,
+                null=True,
+                on_delete=django.db.models.deletion.CASCADE,
+                related_name='visits',
+                to='adventures.location'
+            ),
+        ),
+        # Add transportation FK
+        migrations.AddField(
+            model_name='visit',
+            name='transportation',
+            field=models.ForeignKey(
+                blank=True,
+                null=True,
+                on_delete=django.db.models.deletion.CASCADE,
+                related_name='visits',
+                to='adventures.transportation'
+            ),
+        ),
+        # Add lodging FK
+        migrations.AddField(
+            model_name='visit',
+            name='lodging',
+            field=models.ForeignKey(
+                blank=True,
+                null=True,
+                on_delete=django.db.models.deletion.CASCADE,
+                related_name='visits',
+                to='adventures.lodging'
+            ),
+        ),
+    ]

backend/server/adventures/migrations/0075_visit_transportation_lodging_safe.py

@@ -0,0 +1,74 @@
+# Safe migration for adding Transportation and Lodging to Visit model
+# This migration uses raw SQL to add columns if they don't exist
+
+from django.db import migrations, connection
+
+
+def add_columns_if_not_exist(apps, schema_editor):
+    """Add transportation_id and lodging_id columns to adventures_visit if they don't exist."""
+    with connection.cursor() as cursor:
+        # Check if transportation_id column exists
+        cursor.execute("""
+            SELECT EXISTS (
+                SELECT FROM information_schema.columns
+                WHERE table_name = 'adventures_visit'
+                AND column_name = 'transportation_id'
+            );
+        """)
+        transportation_exists = cursor.fetchone()[0]
+
+        if not transportation_exists:
+            cursor.execute("""
+                ALTER TABLE adventures_visit
+                ADD COLUMN transportation_id UUID NULL
+                REFERENCES adventures_transportation(id) ON DELETE CASCADE;
+            """)
+            cursor.execute("""
+                CREATE INDEX IF NOT EXISTS adventures_visit_transportation_id_idx
+                ON adventures_visit (transportation_id);
+            """)
+
+        # Check if lodging_id column exists
+        cursor.execute("""
+            SELECT EXISTS (
+                SELECT FROM information_schema.columns
+                WHERE table_name = 'adventures_visit'
+                AND column_name = 'lodging_id'
+            );
+        """)
+        lodging_exists = cursor.fetchone()[0]
+
+        if not lodging_exists:
+            cursor.execute("""
+                ALTER TABLE adventures_visit
+                ADD COLUMN lodging_id UUID NULL
+                REFERENCES adventures_lodging(id) ON DELETE CASCADE;
+            """)
+            cursor.execute("""
+                CREATE INDEX IF NOT EXISTS adventures_visit_lodging_id_idx
+                ON adventures_visit (lodging_id);
+            """)
+
+        # Make location_id nullable if it isn't already
+        cursor.execute("""
+            ALTER TABLE adventures_visit
+            ALTER COLUMN location_id DROP NOT NULL;
+        """)
+
+
+def reverse_migration(apps, schema_editor):
+    """Remove the columns (for rollback)."""
+    with connection.cursor() as cursor:
+        cursor.execute("ALTER TABLE adventures_visit DROP COLUMN IF EXISTS transportation_id;")
+        cursor.execute("ALTER TABLE adventures_visit DROP COLUMN IF EXISTS lodging_id;")
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('adventures', '0074_visit_transportation_lodging'),
+    ]
+
+    operations = [
+        migrations.RunPython(add_columns_if_not_exist, reverse_migration),
+    ]

backend/server/adventures/migrations/0076_add_tags_transportation_lodging.py

@@ -0,0 +1,58 @@
+# Safe migration for adding tags field to Transportation and Lodging models
+# This migration uses raw SQL to add columns if they don't exist
+
+from django.db import migrations, connection
+
+
+def add_tags_columns_if_not_exist(apps, schema_editor):
+    """Add tags column to adventures_transportation and adventures_lodging if they don't exist."""
+    with connection.cursor() as cursor:
+        # Check if tags column exists on transportation
+        cursor.execute("""
+            SELECT EXISTS (
+                SELECT FROM information_schema.columns
+                WHERE table_name = 'adventures_transportation'
+                AND column_name = 'tags'
+            );
+        """)
+        transportation_tags_exists = cursor.fetchone()[0]
+
+        if not transportation_tags_exists:
+            cursor.execute("""
+                ALTER TABLE adventures_transportation
+                ADD COLUMN tags VARCHAR(100)[] NULL;
+            """)
+
+        # Check if tags column exists on lodging
+        cursor.execute("""
+            SELECT EXISTS (
+                SELECT FROM information_schema.columns
+                WHERE table_name = 'adventures_lodging'
+                AND column_name = 'tags'
+            );
+        """)
+        lodging_tags_exists = cursor.fetchone()[0]
+
+        if not lodging_tags_exists:
+            cursor.execute("""
+                ALTER TABLE adventures_lodging
+                ADD COLUMN tags VARCHAR(100)[] NULL;
+            """)
+
+
+def reverse_migration(apps, schema_editor):
+    """Remove the columns (for rollback)."""
+    with connection.cursor() as cursor:
+        cursor.execute("ALTER TABLE adventures_transportation DROP COLUMN IF EXISTS tags;")
+        cursor.execute("ALTER TABLE adventures_lodging DROP COLUMN IF EXISTS tags;")
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('adventures', '0075_visit_transportation_lodging_safe'),
+    ]
+
+    operations = [
+        migrations.RunPython(add_tags_columns_if_not_exist, reverse_migration),
+    ]