diff --git a/core/middleware.py b/core/middleware.py
new file mode 100644
index 00000000..c453236b
--- /dev/null
+++ b/core/middleware.py
@@ -0,0 +1,14 @@
+import pghistory
+
+def setup_pghistory_context():
+ """
+ Set up pghistory context middleware to track request information.
+ This function configures what contextual information is stored
+ with each history record.
+ """
+ pghistory.context(lambda request: {
+ 'user': str(request.user) if request.user.is_authenticated else None,
+ 'ip': request.META.get('REMOTE_ADDR'),
+ 'user_agent': request.META.get('HTTP_USER_AGENT'),
+ 'session_key': request.session.session_key
+ })
\ No newline at end of file
diff --git a/core/models.py b/core/models.py
index 0392ad6a..a717d3e2 100644
--- a/core/models.py
+++ b/core/models.py
@@ -2,6 +2,18 @@ from django.db import models
from django.contrib.contenttypes.fields import GenericForeignKey
from django.contrib.contenttypes.models import ContentType
from django.utils.text import slugify
+import pghistory
+
+@pghistory.track()
+class HistoricalModel(models.Model):
+ """
+ Abstract base model that provides universal history tracking via django-pghistory.
+ """
+ class Meta:
+ abstract = True
+
+ def save(self, *args, **kwargs):
+ return super().save(*args, **kwargs)
class SlugHistory(models.Model):
"""
@@ -26,9 +38,11 @@ class SlugHistory(models.Model):
def __str__(self):
return f"Old slug '{self.old_slug}' for {self.content_object}"
-class SluggedModel(models.Model):
+@pghistory.track()
+class SluggedModel(HistoricalModel):
"""
Abstract base model that provides slug functionality with history tracking.
+ Inherits from HistoricalModel to get universal history tracking.
"""
name = models.CharField(max_length=200)
slug = models.SlugField(max_length=200, unique=True)
@@ -55,6 +69,7 @@ class SluggedModel(models.Model):
if not self.slug:
self.slug = slugify(self.name)
+ # Call HistoricalModel's save to ensure history tracking
super().save(*args, **kwargs)
def get_id_field_name(self):
diff --git a/history/apps.py b/history/apps.py
new file mode 100644
index 00000000..95bc91d8
--- /dev/null
+++ b/history/apps.py
@@ -0,0 +1,12 @@
+from django.apps import AppConfig
+
+class HistoryConfig(AppConfig):
+ default_auto_field = 'django.db.models.BigAutoField'
+ name = 'history'
+ verbose_name = 'History Tracking'
+
+ def ready(self):
+ """Initialize app and signal handlers"""
+ from django.dispatch import Signal
+ # Create a signal for history updates
+ self.history_updated = Signal()
\ No newline at end of file
diff --git a/history/templates/history/partials/history_timeline.html b/history/templates/history/partials/history_timeline.html
new file mode 100644
index 00000000..541fcac2
--- /dev/null
+++ b/history/templates/history/partials/history_timeline.html
@@ -0,0 +1,29 @@
+
+
+ {% for event in events %}
+
+
+ {{ event.pgh_label|title }}
+
+
+
+ {% if event.pgh_context.metadata.user %}
+
+
+ {{ event.pgh_context.metadata.user }}
+
+ {% endif %}
+ {% if event.pgh_data %}
+
+
{{ event.pgh_data|pprint }}
+
+ {% endif %}
+
+
+ {% endfor %}
+
+
\ No newline at end of file
diff --git a/history/templatetags/history_tags.py b/history/templatetags/history_tags.py
new file mode 100644
index 00000000..0c1b4177
--- /dev/null
+++ b/history/templatetags/history_tags.py
@@ -0,0 +1,17 @@
+from django import template
+import json
+
+register = template.Library()
+
+@register.filter
+def pprint(value):
+ """Pretty print JSON data"""
+ if isinstance(value, str):
+ try:
+ value = json.loads(value)
+ except json.JSONDecodeError:
+ return value
+
+ if isinstance(value, (dict, list)):
+ return json.dumps(value, indent=2)
+ return str(value)
\ No newline at end of file
diff --git a/history/urls.py b/history/urls.py
new file mode 100644
index 00000000..4382537f
--- /dev/null
+++ b/history/urls.py
@@ -0,0 +1,10 @@
+from django.urls import path
+from .views import HistoryTimelineView
+
+app_name = 'history'
+
+urlpatterns = [
+ path('timeline///',
+ HistoryTimelineView.as_view(),
+ name='timeline'),
+]
\ No newline at end of file
diff --git a/history/views.py b/history/views.py
new file mode 100644
index 00000000..9b4d50b2
--- /dev/null
+++ b/history/views.py
@@ -0,0 +1,41 @@
+from django.views import View
+from django.shortcuts import render
+from django.http import JsonResponse
+from django.contrib.contenttypes.models import ContentType
+import pghistory
+
+def serialize_event(event):
+ """Serialize a history event for JSON response"""
+ return {
+ 'label': event.pgh_label,
+ 'created_at': event.pgh_created_at.isoformat(),
+ 'context': event.pgh_context,
+ 'data': event.pgh_data,
+ }
+
+class HistoryTimelineView(View):
+ """View for displaying object history timeline"""
+
+ def get(self, request, content_type_id, object_id):
+ # Get content type and object
+ content_type = ContentType.objects.get_for_id(content_type_id)
+ obj = content_type.get_object_for_this_type(id=object_id)
+
+ # Get history events
+ events = pghistory.models.Event.objects.filter(
+ pgh_obj_model=content_type.model_class(),
+ pgh_obj_id=object_id
+ ).order_by('-pgh_created_at')[:25]
+
+ context = {
+ 'events': events,
+ 'content_type': content_type,
+ 'object': obj,
+ }
+
+ if request.htmx:
+ return render(request, "history/partials/history_timeline.html", context)
+
+ return JsonResponse({
+ 'history': [serialize_event(e) for e in events]
+ })
\ No newline at end of file
diff --git a/moderation/admin.py b/moderation/admin.py
index 43c377bb..b789b30a 100644
--- a/moderation/admin.py
+++ b/moderation/admin.py
@@ -2,6 +2,8 @@ from django.contrib import admin
from django.contrib.admin import AdminSite
from django.utils.html import format_html
from django.urls import reverse
+from django.utils.safestring import mark_safe
+import pghistory
from .models import EditSubmission, PhotoSubmission
class ModerationAdminSite(AdminSite):
@@ -75,6 +77,33 @@ class PhotoSubmissionAdmin(admin.ModelAdmin):
obj.reject(request.user, obj.notes)
super().save_model(request, obj, form, change)
+class HistoryAdmin(admin.ModelAdmin):
+ """Admin interface for viewing model history events"""
+ list_display = ['pgh_label', 'pgh_created_at', 'get_object_link', 'get_context']
+ list_filter = ['pgh_label', 'pgh_created_at']
+ readonly_fields = ['pgh_label', 'pgh_obj_id', 'pgh_data', 'pgh_context', 'pgh_created_at']
+ date_hierarchy = 'pgh_created_at'
+
+ def get_object_link(self, obj):
+ """Display a link to the related object if possible"""
+ if obj.pgh_obj and hasattr(obj.pgh_obj, 'get_absolute_url'):
+ url = obj.pgh_obj.get_absolute_url()
+ return format_html('{}', url, str(obj.pgh_obj))
+ return str(obj.pgh_obj or '')
+ get_object_link.short_description = 'Object'
+
+ def get_context(self, obj):
+ """Format the context data nicely"""
+ if not obj.pgh_context:
+ return '-'
+ html = ['
']
+ for key, value in obj.pgh_context.items():
+ html.append(f'