feat: Entity system overhaul — visits, types, shared architecture, ratings, pricing

- Expand Visit model: support Transportation/Lodging parents, rating, pricing, collection link
- Add admin-managed types: TransportationType, LodgingType, AdventureType, ActivityType with seed data
- Convert Transportation/Lodging collections from FK to M2M, add tags support
- Add country FK fields to Transportation (origin/dest) and Lodging
- Add cached average_rating to Location, Transportation, Lodging
- Default is_public=True on all entities
- Add 52 shared frontend components (modal, form, cards, detail, list)
- Refactor entity modals, details, and visits to use shared base components
- Add entity list pages with filters, sorting, and public tab
- Update cards with visit count, tags, rating, last visit date
- Add dashboard stats (total km, nights, places)
- Add collection enhancements (public tab, sharing, broken items, recommendations)
- Sync all i18n translations

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: wanoo

.gitignore

@@ -3,13 +3,3 @@
 .vscode/settings.json
 .pnpm-store/
 .env
-
-# Cursor IDE configuration
-.cursor/
-.cursorrules
-
-# Internal development documentation (not for public repo)
-docs/cursor-prompts/
-docs/FIX_PACKAGE_PLAN.md
-docs/GITHUB_COMMENTS.md
-BUGFIX_DOCUMENTATION.md

backend/server/adventures/admin.py

@@ -2,8 +2,8 @@
 from django.contrib import admin
 from django.utils.html import mark_safe, format_html
 from django.urls import reverse
-from .models import Location, Checklist, ChecklistItem, Collection, Transportation, Note, ContentImage, Visit, Category, ContentAttachment, Lodging, CollectionInvite, Trail, Activity, CollectionItineraryItem, CollectionItineraryDay
-from worldtravel.models import Country, Region, VisitedRegion, City, VisitedCity
+from .models import Location, Checklist, ChecklistItem, Collection, Transportation, Note, ContentImage, Visit, Category, ContentAttachment, Lodging, CollectionInvite, Trail, Activity, CollectionItineraryItem, CollectionItineraryDay, TransportationType, LodgingType, AdventureType, ActivityType
+# worldtravel models are now registered in worldtravel/admin.py
 from allauth.account.decorators import secure_admin_login
 
 admin.autodiscover()
@@ -45,38 +45,6 @@ def get_visit_count(self, obj):
     get_visit_count.short_description = 'Visit Count'
 
 
-class CountryAdmin(admin.ModelAdmin):
-    list_display = ('name', 'country_code', 'number_of_regions')
-    list_filter = ('subregion',)
-    search_fields = ('name', 'country_code')
-
-    def number_of_regions(self, obj):
-        return Region.objects.filter(country=obj).count()
-
-    number_of_regions.short_description = 'Number of Regions'
-
-
-class RegionAdmin(admin.ModelAdmin):
-    list_display = ('name', 'country', 'number_of_visits')
-    list_filter = ('country',)
-    search_fields = ('name', 'country__name')
-    # list_filter = ('country', 'number_of_visits')
-
-    def number_of_visits(self, obj):
-        return VisitedRegion.objects.filter(region=obj).count()
-    
-    number_of_visits.short_description = 'Number of Visits'
-
-class CityAdmin(admin.ModelAdmin):
-    list_display = ('name', 'region', 'country')
-    list_filter = ('region', 'region__country')
-    search_fields = ('name', 'region__name', 'region__country__name')
-
-    def country(self, obj):
-        return obj.region.country.name
-
-    country.short_description = 'Country'
-
 from django.contrib import admin
 from django.contrib.auth.admin import UserAdmin
 from users.models import CustomUser
@@ -193,28 +161,55 @@ def object_link(self, obj):
 
     object_link.short_description = 'Item'
 
+class TransportationTypeAdmin(admin.ModelAdmin):
+    list_display = ('key', 'name', 'icon', 'display_order', 'is_active')
+    list_editable = ('name', 'icon', 'display_order', 'is_active')
+    search_fields = ('key', 'name')
+    ordering = ('display_order', 'name')
+
+
+class LodgingTypeAdmin(admin.ModelAdmin):
+    list_display = ('key', 'name', 'icon', 'display_order', 'is_active')
+    list_editable = ('name', 'icon', 'display_order', 'is_active')
+    search_fields = ('key', 'name')
+    ordering = ('display_order', 'name')
+
+
+class AdventureTypeAdmin(admin.ModelAdmin):
+    list_display = ('key', 'name', 'icon', 'display_order', 'is_active')
+    list_editable = ('name', 'icon', 'display_order', 'is_active')
+    search_fields = ('key', 'name')
+    ordering = ('display_order', 'name')
+
+
+class ActivityTypeAdmin(admin.ModelAdmin):
+    list_display = ('key', 'name', 'icon', 'color', 'display_order', 'is_active')
+    list_editable = ('name', 'icon', 'color', 'display_order', 'is_active')
+    search_fields = ('key', 'name')
+    ordering = ('display_order', 'name')
+
+
 admin.site.register(CustomUser, CustomUserAdmin)
 admin.site.register(Location, LocationAdmin)
 admin.site.register(Collection, CollectionAdmin)
 admin.site.register(Visit, VisitAdmin)
-admin.site.register(Country, CountryAdmin)
-admin.site.register(Region, RegionAdmin)
-admin.site.register(VisitedRegion)
 admin.site.register(Transportation)
 admin.site.register(Note)
 admin.site.register(Checklist)
 admin.site.register(ChecklistItem)
 admin.site.register(ContentImage, ContentImageImageAdmin)
 admin.site.register(Category, CategoryAdmin)
-admin.site.register(City, CityAdmin)
-admin.site.register(VisitedCity)
 admin.site.register(ContentAttachment)
 admin.site.register(Lodging)
 admin.site.register(CollectionInvite, CollectionInviteAdmin)
 admin.site.register(Trail)
 admin.site.register(Activity, ActivityAdmin)
 admin.site.register(CollectionItineraryItem, CollectionItineraryItemAdmin)
 admin.site.register(CollectionItineraryDay)
+admin.site.register(TransportationType, TransportationTypeAdmin)
+admin.site.register(LodgingType, LodgingTypeAdmin)
+admin.site.register(AdventureType, AdventureTypeAdmin)
+admin.site.register(ActivityType, ActivityTypeAdmin)
 
 admin.site.site_header = 'AdventureLog Admin'
 admin.site.site_title = 'AdventureLog Admin Site'

backend/server/adventures/management/commands/sync_visits_to_worldtravel.py

@@ -0,0 +1,155 @@
+"""
+Django management command to synchronize visited regions/cities from Visit records.
+
+This command processes all visits and marks their corresponding cities/regions as visited
+using reverse geocoding of the parent item's coordinates.
+
+Usage:
+    python manage.py sync_visits_to_worldtravel
+    python manage.py sync_visits_to_worldtravel --dry-run
+    python manage.py sync_visits_to_worldtravel --user-id 123
+"""
+
+from django.core.management.base import BaseCommand, CommandError
+from django.contrib.auth import get_user_model
+from adventures.models import Visit
+from adventures.signals import mark_city_region_visited, _get_visit_coordinates
+import logging
+
+logger = logging.getLogger(__name__)
+User = get_user_model()
+
+
+class Command(BaseCommand):
+    help = 'Sync visited regions/cities from Visit records using reverse geocoding'
+
+    def add_arguments(self, parser):
+        parser.add_argument(
+            '--dry-run',
+            action='store_true',
+            help='Show what would be created without making changes',
+        )
+        parser.add_argument(
+            '--user-id',
+            type=int,
+            help='Sync only for a specific user ID',
+        )
+        parser.add_argument(
+            '--verbose',
+            action='store_true',
+            help='Show detailed output for each visit',
+        )
+
+    def handle(self, *args, **options):
+        dry_run = options['dry_run']
+        user_id = options.get('user_id')
+        verbose = options['verbose']
+
+        if dry_run:
+            self.stdout.write(
+                self.style.WARNING('DRY RUN MODE - No changes will be made')
+            )
+
+        # Build visits queryset
+        visits_queryset = Visit.objects.select_related(
+            'user', 'location', 'transportation', 'lodging'
+        ).filter(user__isnull=False)
+
+        if user_id:
+            visits_queryset = visits_queryset.filter(user_id=user_id)
+
+        try:
+            total_visits = visits_queryset.count()
+        except Exception as e:
+            # May fail during migration if schema is changing
+            self.stdout.write(self.style.WARNING(f'Could not count visits (migration in progress?): {e}'))
+            return
+
+        if total_visits == 0:
+            self.stdout.write(self.style.WARNING('No visits found'))
+            return
+
+        self.stdout.write(f'Processing {total_visits} visit(s)...\n')
+
+        # Track statistics
+        total_cities_marked = 0
+        total_regions_marked = 0
+        visits_processed = 0
+        visits_with_coords = 0
+        visits_with_changes = 0
+
+        try:
+            visits_list = list(visits_queryset)
+        except Exception as e:
+            # May fail during migration if schema is changing
+            self.stdout.write(self.style.WARNING(f'Could not load visits (migration in progress?): {e}'))
+            return
+
+        for visit in visits_list:
+            visits_processed += 1
+
+            lat, lon = _get_visit_coordinates(visit)
+
+            if lat is None or lon is None:
+                if verbose:
+                    self.stdout.write(f'  Visit {visit.id}: No coordinates')
+                continue
+
+            visits_with_coords += 1
+
+            if dry_run:
+                if verbose:
+                    self.stdout.write(
+                        f'  Visit {visit.id}: Would geocode ({lat}, {lon}) for user {visit.user.username}'
+                    )
+                continue
+
+            try:
+                city_marked, region_marked = mark_city_region_visited(
+                    visit.user, lat, lon
+                )
+
+                if city_marked:
+                    total_cities_marked += 1
+                if region_marked:
+                    total_regions_marked += 1
+                if city_marked or region_marked:
+                    visits_with_changes += 1
+
+                if verbose and (city_marked or region_marked):
+                    self.stdout.write(
+                        f'  Visit {visit.id}: city={city_marked}, region={region_marked}'
+                    )
+
+            except Exception as e:
+                self.stdout.write(
+                    self.style.ERROR(f'Error processing visit {visit.id}: {str(e)}')
+                )
+                logger.exception(f'Error processing visit {visit.id}')
+
+            # Progress indicator
+            if visits_processed % 100 == 0:
+                self.stdout.write(f'Processed {visits_processed}/{total_visits} visits...')
+
+        # Summary
+        self.stdout.write('\n' + '=' * 60)
+        if dry_run:
+            self.stdout.write(
+                self.style.SUCCESS(
+                    f'DRY RUN COMPLETE:\n'
+                    f'  Visits processed: {visits_processed}\n'
+                    f'  Visits with coordinates: {visits_with_coords}\n'
+                    f'  Would geocode {visits_with_coords} visits'
+                )
+            )
+        else:
+            self.stdout.write(
+                self.style.SUCCESS(
+                    f'SYNC COMPLETE:\n'
+                    f'  Visits processed: {visits_processed}\n'
+                    f'  Visits with coordinates: {visits_with_coords}\n'
+                    f'  Visits with changes: {visits_with_changes}\n'
+                    f'  New regions marked: {total_regions_marked}\n'
+                    f'  New cities marked: {total_cities_marked}'
+                )
+            )