django-cfg 1.2.29__py3-none-any.whl → 1.3.1__py3-none-any.whl

django_cfg/apps/payments/admin/balance_admin.py

@@ -1,40 +1,68 @@
 """
-Admin interfaces for balance and transaction management.
+Balance Admin interfaces with Unfold integration.
+
+Advanced balance and transaction management with bulk operations.
 """
 
 from django.contrib import admin
 from django.utils.html import format_html
-from django.contrib.humanize.templatetags.humanize import naturaltime
-from django.urls import reverse
+from django.contrib.humanize.templatetags.humanize import naturaltime, intcomma
+from django.contrib import messages
 from django.shortcuts import redirect
+from django.utils.safestring import mark_safe
+from django.db.models import Count, Sum, Q, Avg
+from django.utils import timezone
+from datetime import timedelta
+from decimal import Decimal
+from typing import Optional
+
 from unfold.admin import ModelAdmin
 from unfold.decorators import display, action
 from unfold.enums import ActionVariant
 
 from ..models import UserBalance, Transaction
-from .filters import BalanceRangeFilter, TransactionTypeFilter, UserEmailFilter, RecentActivityFilter
+from .filters import BalanceRangeFilter, RecentActivityFilter
+from django_cfg.modules.django_logger import get_logger
+
+logger = get_logger("balance_admin")
 
 
 @admin.register(UserBalance)
 class UserBalanceAdmin(ModelAdmin):
-    """Admin interface for user balances."""
+    """
+    Advanced UserBalance admin with bulk operations and financial monitoring.
+    
+    Features:
+    - Balance range filtering and visualization
+    - Bulk balance adjustments with audit trail
+    - Financial statistics and alerts
+    - Transaction history integration
+    - Security features for balance modifications
+    """
+    
+    # Custom template for balance statistics
+    change_list_template = 'admin/payments/balance/change_list.html'
     
     list_display = [
         'user_display',
         'balance_display',
-        'reserved_display',
-        'available_display',
-        'last_transaction_display',
+        'balance_status',
+        'transaction_count_display',
+        'last_activity_display',
         'created_at_display'
     ]
     
     list_display_links = ['user_display']
     
-    search_fields = ['user__email', 'user__first_name', 'user__last_name']
+    search_fields = [
+        'user__email',
+        'user__first_name',
+        'user__last_name',
+        'user__username'
+    ]
     
     list_filter = [
         BalanceRangeFilter,
-        UserEmailFilter,
         RecentActivityFilter,
         'created_at'
     ]
@@ -42,8 +70,16 @@ class UserBalanceAdmin(ModelAdmin):
     readonly_fields = [
         'created_at',
         'updated_at',
-        'transaction_history',
-        'balance_statistics'
+        'last_transaction_at'
+    ]
+    
+    # Unfold actions
+    actions_list = [
+        'add_funds_bulk',
+        'subtract_funds_bulk',
+        'reset_zero_balances',
+        'export_balance_report',
+        'send_low_balance_alerts'
     ]
     
     fieldsets = [
@@ -51,15 +87,15 @@ class UserBalanceAdmin(ModelAdmin):
             'fields': ['user']
         }),
         ('Balance Details', {
-            'fields': ['amount_usd', 'reserved_usd']
-        }),
-        ('Statistics', {
-            'fields': ['balance_statistics'],
-            'classes': ['collapse']
+            'fields': [
+                'balance_usd',
+                'reserved_usd'
+            ]
         }),
-        ('Transaction History', {
-            'fields': ['transaction_history'],
-            'classes': ['collapse']
+        ('Activity Tracking', {
+            'fields': [
+                'last_transaction_at'
+            ]
         }),
         ('Timestamps', {
             'fields': ['created_at', 'updated_at'],
@@ -67,368 +103,627 @@ class UserBalanceAdmin(ModelAdmin):
         })
     ]
     
-    actions_detail = ['add_funds', 'view_transactions']
+    def get_queryset(self, request):
+        """Optimize queryset with user data and transaction counts."""
+        return super().get_queryset(request).select_related('user').annotate(
+            transaction_count=Count('user__transaction_set')
+        )
     
-    @display(description="User")
+    @display(description="User", ordering='user__email')
     def user_display(self, obj):
-        """Display user with avatar."""
-        user = obj.user
-        if hasattr(user, 'avatar') and user.avatar:
-            avatar_html = f'<img src="{user.avatar.url}" style="width: 24px; height: 24px; border-radius: 50%; margin-right: 8px;" />'
+        """Display user information with avatar and details."""
+        if obj.user:
+            display_name = obj.user.get_full_name() or obj.user.username
+            
+            # Determine user tier based on balance
+            if obj.balance_usd >= 1000:
+                tier_icon = "🐋"
+                tier_color = "text-purple-600"
+                tier_name = "Whale"
+            elif obj.balance_usd >= 100:
+                tier_icon = "💎"
+                tier_color = "text-blue-600"
+                tier_name = "Premium"
+            elif obj.balance_usd >= 10:
+                tier_icon = "💰"
+                tier_color = "text-green-600"
+                tier_name = "Active"
+            elif obj.balance_usd > 0:
+                tier_icon = "🪙"
+                tier_color = "text-yellow-600"
+                tier_name = "Basic"
+            else:
+                tier_icon = "💸"
+                tier_color = "text-gray-600"
+                tier_name = "Empty"
+            
+            return format_html(
+                '<div class="flex items-center space-x-3">'
+                '<div class="w-10 h-10 bg-gradient-to-br from-blue-500 to-purple-600 rounded-full flex items-center justify-center text-white text-sm font-bold">'
+                '{}'
+                '</div>'
+                '<div>'
+                '<div class="font-medium text-gray-900 dark:text-gray-100">{}</div>'
+                '<div class="text-xs text-gray-500">{}</div>'
+                '<div class="text-xs {}"><span class="mr-1">{}</span>{}</div>'
+                '</div>'
+                '</div>',
+                display_name[0].upper() if display_name else 'U',
+                display_name,
+                obj.user.email,
+                tier_color,
+                tier_icon,
+                tier_name
+            )
+        return format_html('<span class="text-gray-500">No user</span>')
+    
+    @display(description="Balance", ordering='balance_usd')
+    def balance_display(self, obj):
+        """Display balance with visual indicators and reserved amounts."""
+        balance = obj.balance_usd
+        reserved = obj.reserved_usd or 0
+        available = balance - reserved
+        
+        # Color coding based on balance
+        if balance < 0:
+            balance_color = "text-red-600 dark:text-red-400"
+            balance_icon = "⚠️"
+        elif balance == 0:
+            balance_color = "text-gray-600 dark:text-gray-400"
+            balance_icon = "💸"
+        elif balance < 10:
+            balance_color = "text-yellow-600 dark:text-yellow-400"
+            balance_icon = "🪙"
+        elif balance < 100:
+            balance_color = "text-green-600 dark:text-green-400"
+            balance_icon = "💰"
         else:
-            initials = f"{user.first_name[:1]}{user.last_name[:1]}" if user.first_name and user.last_name else user.email[:2]
-            avatar_html = f'<div style="width: 24px; height: 24px; border-radius: 50%; background: #6c757d; color: white; display: inline-flex; align-items: center; justify-content: center; font-size: 10px; margin-right: 8px;">{initials.upper()}</div>'
+            balance_color = "text-blue-600 dark:text-blue-400"
+            balance_icon = "💎"
         
-        return format_html(
-            '{}<strong>{}</strong><br><small>{}</small>',
-            avatar_html,
-            user.get_full_name() or user.email,
-            user.email
-        )
+        html = f'''
+        <div class="text-right">
+            <div class="font-bold text-lg {balance_color}">
+                <span class="mr-1">{balance_icon}</span>${balance:,.2f}
+            </div>
+        '''
+        
+        if reserved > 0:
+            html += f'''
+            <div class="text-xs text-orange-600 dark:text-orange-400">
+                Reserved: ${reserved:,.2f}
+            </div>
+            <div class="text-xs text-gray-500">
+                Available: ${available:,.2f}
+            </div>
+            '''
+        
+        html += '</div>'
+        
+        return format_html(html)
     
-    @display(description="Balance")
-    def balance_display(self, obj):
-        """Display balance with color coding."""
-        amount = obj.amount_usd
-        if amount > 100:
-            color = '#28a745'  # Green
-        elif amount > 10:
-            color = '#ffc107'  # Yellow
-        elif amount > 0:
-            color = '#fd7e14'  # Orange
+    @display(description="Status")
+    def balance_status(self, obj):
+        """Display balance status with alerts."""
+        balance = obj.balance_usd
+        reserved = obj.reserved_usd or 0
+        
+        badges = []
+        
+        if balance < 0:
+            badges.append('<span class="inline-flex items-center rounded-full bg-red-100 px-2 py-0.5 text-xs font-medium text-red-800 dark:bg-red-900 dark:text-red-200">⚠️ Negative</span>')
+        elif balance == 0:
+            badges.append('<span class="inline-flex items-center rounded-full bg-gray-100 px-2 py-0.5 text-xs font-medium text-gray-800 dark:bg-gray-900 dark:text-gray-200">💸 Empty</span>')
+        elif balance < 1:
+            badges.append('<span class="inline-flex items-center rounded-full bg-yellow-100 px-2 py-0.5 text-xs font-medium text-yellow-800 dark:bg-yellow-900 dark:text-yellow-200">⚠️ Low</span>')
         else:
-            color = '#dc3545'  # Red
+            badges.append('<span class="inline-flex items-center rounded-full bg-green-100 px-2 py-0.5 text-xs font-medium text-green-800 dark:bg-green-900 dark:text-green-200">✅ Active</span>')
         
-        return format_html(
-            '<span style="color: {}; font-weight: bold;">${:.2f}</span>',
-            color, amount
-        )
+        if reserved > 0:
+            badges.append('<span class="inline-flex items-center rounded-full bg-orange-100 px-2 py-0.5 text-xs font-medium text-orange-800 dark:bg-orange-900 dark:text-orange-200">🔒 Reserved</span>')
+        
+        return format_html('<div class="space-y-1">{}</div>', ''.join(badges))
     
-    @display(description="Reserved")
-    def reserved_display(self, obj):
-        """Display reserved amount."""
-        if obj.reserved_usd > 0:
+    @display(description="Transactions")
+    def transaction_count_display(self, obj):
+        """Display transaction count and recent activity."""
+        count = getattr(obj, 'transaction_count', 0)
+        
+        if count > 0:
+            # Get recent transaction count (last 7 days)
+            recent_threshold = timezone.now() - timedelta(days=7)
+            recent_count = Transaction.objects.filter(
+                user=obj.user,
+                created_at__gte=recent_threshold
+            ).count()
+            
             return format_html(
-                '<span style="color: #6c757d;">${:.2f}</span>',
-                obj.reserved_usd
+                '<div class="text-center">'
+                '<div class="font-bold text-blue-600 dark:text-blue-400">{}</div>'
+                '<div class="text-xs text-gray-500">total</div>'
+                '{}'
+                '</div>',
+                count,
+                f'<div class="text-xs text-green-600 dark:text-green-400">{recent_count} recent</div>' if recent_count > 0 else ''
             )
-        return "—"
-    
-    @display(description="Available")
-    def available_display(self, obj):
-        """Display available balance."""
-        available = obj.amount_usd - obj.reserved_usd
+        
         return format_html(
-            '<span style="font-weight: bold;">${:.2f}</span>',
-            available
+            '<div class="text-center text-gray-500">'
+            '<div>0</div>'
+            '<div class="text-xs">No transactions</div>'
+            '</div>'
         )
     
-    @display(description="Last Transaction")
-    def last_transaction_display(self, obj):
-        """Display last transaction."""
-        last_transaction = obj.user.transactions.order_by('-created_at').first()
-        if last_transaction:
+    @display(description="Last Activity", ordering='last_transaction_at')
+    def last_activity_display(self, obj):
+        """Display last transaction activity."""
+        if obj.last_transaction_at:
+            time_ago = timezone.now() - obj.last_transaction_at
+            
+            if time_ago < timedelta(hours=1):
+                color = "text-green-600 dark:text-green-400"
+                icon = "🟢"
+            elif time_ago < timedelta(days=1):
+                color = "text-yellow-600 dark:text-yellow-400"
+                icon = "🟡"
+            elif time_ago < timedelta(days=7):
+                color = "text-orange-600 dark:text-orange-400"
+                icon = "🟠"
+            else:
+                color = "text-red-600 dark:text-red-400"
+                icon = "🔴"
+            
             return format_html(
-                '<span style="color: {};">{} ${:.2f}</span><br><small>{}</small>',
-                '#28a745' if last_transaction.amount_usd > 0 else '#dc3545',
-                '+' if last_transaction.amount_usd > 0 else '',
-                abs(last_transaction.amount_usd),
-                naturaltime(last_transaction.created_at)
+                '<div class="text-xs {}">'
+                '<span class="mr-1">{}</span>{}'
+                '</div>',
+                color,
+                icon,
+                naturaltime(obj.last_transaction_at)
             )
-        return "No transactions"
+        
+        return format_html(
+            '<div class="text-xs text-gray-500">Never</div>'
+        )
     
-    @display(description="Created")
+    @display(description="Created", ordering='created_at')
     def created_at_display(self, obj):
         """Display creation date."""
-        return naturaltime(obj.created_at)
-    
-    def balance_statistics(self, obj):
-        """Show balance statistics."""
-        transactions = obj.user.transactions.all()
-        total_credited = sum(t.amount_usd for t in transactions if t.amount_usd > 0)
-        total_debited = sum(abs(t.amount_usd) for t in transactions if t.amount_usd < 0)
-        transaction_count = transactions.count()
-        
         return format_html(
-            '<div style="line-height: 1.6;">'
-            '<strong>Statistics:</strong><br>'
-            '• Total Credited: <span style="color: #28a745;">${:.2f}</span><br>'
-            '• Total Debited: <span style="color: #dc3545;">${:.2f}</span><br>'
-            '• Net Balance: <span style="color: {};">${:.2f}</span><br>'
-            '• Total Transactions: {}<br>'
-            '• Available Balance: <strong>${:.2f}</strong>'
+            '<div class="text-xs">'
+            '<div>{}</div>'
+            '<div class="text-gray-500">{}</div>'
             '</div>',
-            total_credited,
-            total_debited,
-            '#28a745' if (total_credited - total_debited) > 0 else '#dc3545',
-            total_credited - total_debited,
-            transaction_count,
-            obj.amount_usd - obj.reserved_usd
+            obj.created_at.strftime('%Y-%m-%d'),
+            naturaltime(obj.created_at)
         )
     
-    balance_statistics.short_description = "Balance Statistics"
+    def changelist_view(self, request, extra_context=None):
+        """Add balance statistics to changelist context."""
+        extra_context = extra_context or {}
+        
+        try:
+            # Basic statistics
+            total_balances = UserBalance.objects.count()
+            
+            # Balance statistics
+            balance_stats = UserBalance.objects.aggregate(
+                total_balance=Sum('balance_usd'),
+                avg_balance=Avg('balance_usd'),
+                total_reserved=Sum('reserved_usd')
+            )
+            
+            # Balance distribution
+            zero_balances = UserBalance.objects.filter(balance_usd=0).count()
+            negative_balances = UserBalance.objects.filter(balance_usd__lt=0).count()
+            low_balances = UserBalance.objects.filter(balance_usd__gt=0, balance_usd__lt=10).count()
+            medium_balances = UserBalance.objects.filter(balance_usd__gte=10, balance_usd__lt=100).count()
+            high_balances = UserBalance.objects.filter(balance_usd__gte=100, balance_usd__lt=1000).count()
+            whale_balances = UserBalance.objects.filter(balance_usd__gte=1000).count()
+            
+            # Recent activity
+            recent_threshold = timezone.now() - timedelta(days=7)
+            active_balances = UserBalance.objects.filter(
+                last_transaction_at__gte=recent_threshold
+            ).count()
+            
+            # Top balances
+            top_balances = UserBalance.objects.filter(
+                balance_usd__gt=0
+            ).order_by('-balance_usd')[:5]
+            
+            extra_context.update({
+                'balance_stats': {
+                    'total_balances': total_balances,
+                    'total_balance': balance_stats['total_balance'] or 0,
+                    'avg_balance': balance_stats['avg_balance'] or 0,
+                    'total_reserved': balance_stats['total_reserved'] or 0,
+                    'zero_balances': zero_balances,
+                    'negative_balances': negative_balances,
+                    'low_balances': low_balances,
+                    'medium_balances': medium_balances,
+                    'high_balances': high_balances,
+                    'whale_balances': whale_balances,
+                    'active_balances': active_balances,
+                    'top_balances': top_balances,
+                }
+            })
+            
+        except Exception as e:
+            logger.warning(f"Failed to generate balance statistics: {e}")
+            extra_context['balance_stats'] = None
+        
+        return super().changelist_view(request, extra_context)
+    
+    # ===== ADMIN ACTIONS =====
     
-    def transaction_history(self, obj):
-        """Show recent transaction history."""
-        transactions = obj.user.transactions.order_by('-created_at')[:10]
+    @action(
+        description="💰 Add Funds (Bulk)",
+        icon="add_circle",
+        variant=ActionVariant.SUCCESS
+    )
+    def add_funds_bulk(self, request, queryset):
+        """Add funds to selected user balances."""
         
-        if not transactions:
-            return "No transactions"
+        # This would typically show a form for amount input
+        # For now, we'll add a fixed amount as an example
+        amount = Decimal('10.00')  # In production, this should come from a form
         
-        html = '<div style="line-height: 1.8;">'
-        for transaction in transactions:
-            amount_color = '#28a745' if transaction.amount_usd > 0 else '#dc3545'
-            amount_sign = '+' if transaction.amount_usd > 0 else ''
-            
-            html += f'''
-            <div style="border-bottom: 1px solid #eee; padding: 4px 0;">
-                <span style="color: {amount_color}; font-weight: bold;">
-                    {amount_sign}${abs(transaction.amount_usd):.2f}
-                </span>
-                <span style="margin-left: 10px; color: #6c757d;">
-                    {transaction.get_transaction_type_display()}
-                </span>
-                <br>
-                <small style="color: #999;">
-                    {transaction.description[:50]}{'...' if len(transaction.description) > 50 else ''}
-                    • {naturaltime(transaction.created_at)}
-                </small>
-            </div>
-            '''
+        updated_count = 0
         
-        if obj.user.transactions.count() > 10:
-            html += f'<p><small><em>... and {obj.user.transactions.count() - 10} more transactions</em></small></p>'
+        for balance in queryset:
+            try:
+                # Use manager method for proper transaction handling
+                UserBalance.objects.add_funds_to_user(
+                    user=balance.user,
+                    amount=amount,
+                    transaction_type='admin_adjustment',
+                    description=f'Bulk funds addition by admin {request.user.username}'
+                )
+                updated_count += 1
+                
+            except Exception as e:
+                logger.error(f"Failed to add funds to user {balance.user.id}: {e}")
         
-        html += '</div>'
-        return format_html(html)
+        if updated_count > 0:
+            messages.success(
+                request,
+                f"💰 Added ${amount} to {updated_count} user balances"
+            )
+            messages.info(
+                request,
+                "ℹ️ All transactions have been logged for audit purposes"
+            )
     
-    transaction_history.short_description = "Recent Transactions"
+    @action(
+        description="💸 Subtract Funds (Bulk)",
+        icon="remove_circle",
+        variant=ActionVariant.WARNING
+    )
+    def subtract_funds_bulk(self, request, queryset):
+        """Subtract funds from selected user balances."""
+        
+        amount = Decimal('5.00')  # In production, this should come from a form
+        
+        updated_count = 0
+        insufficient_funds = 0
+        
+        for balance in queryset:
+            try:
+                if balance.balance_usd >= amount:
+                    UserBalance.objects.subtract_funds_from_user(
+                        user=balance.user,
+                        amount=amount,
+                        transaction_type='admin_adjustment',
+                        description=f'Bulk funds subtraction by admin {request.user.username}'
+                    )
+                    updated_count += 1
+                else:
+                    insufficient_funds += 1
+                    
+            except Exception as e:
+                logger.error(f"Failed to subtract funds from user {balance.user.id}: {e}")
+        
+        if updated_count > 0:
+            messages.success(
+                request,
+                f"💸 Subtracted ${amount} from {updated_count} user balances"
+            )
+        
+        if insufficient_funds > 0:
+            messages.warning(
+                request,
+                f"⚠️ Skipped {insufficient_funds} users with insufficient funds"
+            )
     
     @action(
-        description="💰 Add Funds",
-        icon="attach_money",
-        variant=ActionVariant.SUCCESS
+        description="🔄 Reset Zero Balances",
+        icon="refresh",
+        variant=ActionVariant.INFO
     )
-    def add_funds(self, request, object_id):
-        """Add funds to user balance."""
-        # In real implementation, this would redirect to a custom form
-        balance = self.get_object(request, object_id)
-        if balance:
-            self.message_user(
+    def reset_zero_balances(self, request, queryset):
+        """Reset zero balances and clear reserved amounts."""
+        
+        zero_balances = queryset.filter(balance_usd=0)
+        reset_count = 0
+        
+        for balance in zero_balances:
+            if balance.reserved_usd and balance.reserved_usd > 0:
+                balance.reserved_usd = 0
+                balance.save(update_fields=['reserved_usd'])
+                reset_count += 1
+        
+        if reset_count > 0:
+            messages.success(
                 request,
-                f"Add funds form would open for {balance.user.email}",
-                level='info'
+                f"🔄 Reset reserved amounts for {reset_count} zero balances"
+            )
+        else:
+            messages.info(
+                request,
+                "ℹ️ No zero balances with reserved amounts found"
             )
-        return redirect(request.META.get('HTTP_REFERER', '/admin/'))
     
     @action(
-        description="📊 View Transactions",
-        icon="receipt_long",
+        description="📊 Export Balance Report",
+        icon="download",
         variant=ActionVariant.INFO
     )
-    def view_transactions(self, request, object_id):
-        """View all transactions for this user."""
-        balance = self.get_object(request, object_id)
-        if balance:
-            url = reverse('admin:django_cfg_payments_transaction_changelist')
-            return redirect(f"{url}?user__id__exact={balance.user.id}")
-        return redirect(request.META.get('HTTP_REFERER', '/admin/'))
+    def export_balance_report(self, request, queryset):
+        """Export balance report to CSV."""
+        
+        import csv
+        from django.http import HttpResponse
+        
+        response = HttpResponse(content_type='text/csv')
+        response['Content-Disposition'] = f'attachment; filename="balance_report_{timezone.now().strftime("%Y%m%d_%H%M%S")}.csv"'
+        
+        writer = csv.writer(response)
+        writer.writerow([
+            'User Email', 'User Name', 'Balance USD', 'Reserved USD', 'Available USD',
+            'Last Transaction', 'Created', 'Status'
+        ])
+        
+        for balance in queryset:
+            available = balance.balance_usd - (balance.reserved_usd or 0)
+            
+            if balance.balance_usd < 0:
+                status = 'Negative'
+            elif balance.balance_usd == 0:
+                status = 'Empty'
+            elif balance.balance_usd < 10:
+                status = 'Low'
+            elif balance.balance_usd < 100:
+                status = 'Medium'
+            else:
+                status = 'High'
+            
+            writer.writerow([
+                balance.user.email if balance.user else '',
+                balance.user.get_full_name() if balance.user else '',
+                balance.balance_usd,
+                balance.reserved_usd or 0,
+                available,
+                balance.last_transaction_at.isoformat() if balance.last_transaction_at else '',
+                balance.created_at.isoformat(),
+                status
+            ])
+        
+        messages.success(
+            request,
+            f"📊 Exported {queryset.count()} balance records to CSV"
+        )
+        
+        return response
+    
+    @action(
+        description="🔔 Send Low Balance Alerts",
+        icon="notifications",
+        variant=ActionVariant.WARNING
+    )
+    def send_low_balance_alerts(self, request, queryset):
+        """Send alerts for low balance users."""
+        
+        low_balance_users = queryset.filter(
+            balance_usd__gt=0,
+            balance_usd__lt=10
+        )
+        
+        alert_count = 0
+        
+        for balance in low_balance_users:
+            try:
+                # In production, this would send an actual notification
+                # For now, we'll just log it
+                logger.info(
+                    f"Low balance alert for user {balance.user.email}: ${balance.balance_usd}"
+                )
+                alert_count += 1
+                
+            except Exception as e:
+                logger.error(f"Failed to send alert to user {balance.user.id}: {e}")
+        
+        if alert_count > 0:
+            messages.success(
+                request,
+                f"🔔 Sent low balance alerts to {alert_count} users"
+            )
+        else:
+            messages.info(
+                request,
+                "ℹ️ No users with low balances found in selection"
+            )
 
 
 @admin.register(Transaction)
 class TransactionAdmin(ModelAdmin):
-    """Admin interface for transactions."""
+    """
+    Transaction admin with detailed tracking and audit capabilities.
+    
+    Features:
+    - Comprehensive transaction history
+    - Financial audit trail
+    - Transaction type filtering
+    - Balance impact visualization
+    """
     
     list_display = [
-        'transaction_display',
+        'transaction_id_display',
         'user_display',
+        'transaction_type_badge',
         'amount_display',
-        'type_display',
-        'payment_display',
-        'subscription_display',
+        'balance_impact_display',
+        'payment_link_display',
         'created_at_display'
     ]
     
-    list_display_links = ['transaction_display']
+    list_display_links = ['transaction_id_display']
     
     search_fields = [
+        'id',
         'user__email',
+        'user__username',
         'description',
-        'payment__internal_payment_id',
-        'subscription__endpoint_group__name'
+        'payment_id'
     ]
     
     list_filter = [
-        TransactionTypeFilter,
-        UserEmailFilter,
+        'transaction_type',
         RecentActivityFilter,
-        'payment__status',
-        'subscription__status',
         'created_at'
     ]
     
     readonly_fields = [
-        'created_at',
-        'transaction_details',
-        'related_objects'
+        'id',
+        'user',
+        'transaction_type',
+        'amount_usd',
+        'balance_after',
+        'payment_id',
+        'description',
+        'created_at'
     ]
     
-    fieldsets = [
-        ('Transaction Information', {
-            'fields': ['user', 'transaction_type', 'amount_usd', 'description']
-        }),
-        ('Related Objects', {
-            'fields': ['payment', 'subscription'],
-            'classes': ['collapse']
-        }),
-        ('Additional Data', {
-            'fields': ['metadata', 'related_objects'],
-            'classes': ['collapse']
-        }),
-        ('Transaction Details', {
-            'fields': ['transaction_details'],
-            'classes': ['collapse']
-        }),
-        ('Timestamps', {
-            'fields': ['created_at'],
-            'classes': ['collapse']
-        })
-    ]
+    def has_add_permission(self, request):
+        """Disable adding transactions through admin (should be created by system)."""
+        return False
+    
+    def has_change_permission(self, request, obj=None):
+        """Disable changing transactions (audit trail integrity)."""
+        return False
     
-    @display(description="Transaction")
-    def transaction_display(self, obj):
-        """Display transaction ID and description."""
+    def has_delete_permission(self, request, obj=None):
+        """Disable deleting transactions (audit trail integrity)."""
+        return False
+    
+    @display(description="Transaction ID", ordering='id')
+    def transaction_id_display(self, obj):
+        """Display transaction ID."""
+        short_id = str(obj.id)[:8]
         return format_html(
-            '<strong>#{}</strong><br><small>{}</small>',
-            str(obj.id)[:8],
-            obj.description[:40] + '...' if len(obj.description) > 40 else obj.description
+            '<span class="font-mono text-sm bg-gray-100 dark:bg-gray-800 px-2 py-1 rounded" '
+            'title="Full ID: {}">{}</span>',
+            obj.id,
+            short_id
         )
     
-    @display(description="User")
+    @display(description="User", ordering='user__email')
     def user_display(self, obj):
         """Display user information."""
+        if obj.user:
+            return format_html(
+                '<div>'
+                '<div class="font-medium">{}</div>'
+                '<div class="text-xs text-gray-500">{}</div>'
+                '</div>',
+                obj.user.get_full_name() or obj.user.username,
+                obj.user.email
+            )
+        return format_html('<span class="text-gray-500">No user</span>')
+    
+    @display(description="Type", ordering='transaction_type')
+    def transaction_type_badge(self, obj):
+        """Display transaction type with colored badge."""
+        type_config = {
+            'payment': ('💳', 'bg-green-100 text-green-800 dark:bg-green-900 dark:text-green-200', 'Payment'),
+            'deposit': ('💰', 'bg-blue-100 text-blue-800 dark:bg-blue-900 dark:text-blue-200', 'Deposit'),
+            'withdrawal': ('💸', 'bg-red-100 text-red-800 dark:bg-red-900 dark:text-red-200', 'Withdrawal'),
+            'refund': ('↩️', 'bg-purple-100 text-purple-800 dark:bg-purple-900 dark:text-purple-200', 'Refund'),
+            'admin_adjustment': ('⚙️', 'bg-orange-100 text-orange-800 dark:bg-orange-900 dark:text-orange-200', 'Admin'),
+            'fee': ('📋', 'bg-gray-100 text-gray-800 dark:bg-gray-900 dark:text-gray-200', 'Fee'),
+        }
+        
+        icon, color_class, label = type_config.get(
+            obj.transaction_type,
+            ('❓', 'bg-gray-100 text-gray-800', obj.transaction_type.title())
+        )
+        
         return format_html(
-            '<strong>{}</strong><br><small>{}</small>',
-            obj.user.get_full_name() or obj.user.email,
-            obj.user.email
+            '<span class="inline-flex items-center rounded-full px-2.5 py-0.5 text-xs font-medium {}">'
+            '{} {}'
+            '</span>',
+            color_class,
+            icon,
+            label
         )
     
-    @display(description="Amount")
+    @display(description="Amount", ordering='amount_usd')
     def amount_display(self, obj):
-        """Display amount with color coding."""
+        """Display transaction amount with sign."""
         amount = obj.amount_usd
-        color = '#28a745' if amount > 0 else '#dc3545'
-        sign = '+' if amount > 0 else ''
         
-        return format_html(
-            '<span style="color: {}; font-weight: bold; font-size: 14px;">{}</span>',
-            color,
-            f'{sign}${abs(amount):.2f}'
-        )
+        if amount > 0:
+            return format_html(
+                '<span class="font-bold text-green-600 dark:text-green-400">+${:,.2f}</span>',
+                amount
+            )
+        elif amount < 0:
+            return format_html(
+                '<span class="font-bold text-red-600 dark:text-red-400">-${:,.2f}</span>',
+                abs(amount)
+            )
+        else:
+            return format_html(
+                '<span class="font-bold text-gray-600 dark:text-gray-400">${:,.2f}</span>',
+                amount
+            )
     
-    @display(description="Type")
-    def type_display(self, obj):
-        """Display transaction type with badge."""
-        type_colors = {
-            'credit': '#28a745',
-            'debit': '#dc3545',
-            'refund': '#17a2b8',
-            'withdrawal': '#ffc107',
-        }
-        
-        color = type_colors.get(obj.transaction_type, '#6c757d')
+    @display(description="Balance Impact")
+    def balance_impact_display(self, obj):
+        """Display balance before/after transaction."""
+        # Calculate balance_before from balance_after and amount_usd
+        balance_before = obj.balance_after - obj.amount_usd
         
         return format_html(
-            '<span style="background: {}; color: white; padding: 2px 6px; border-radius: 3px; font-size: 11px;">{}</span>',
-            color,
-            obj.get_transaction_type_display()
+            '<div class="text-xs">'
+            '<div>Before: <span class="font-mono">${:,.2f}</span></div>'
+            '<div>After: <span class="font-mono">${:,.2f}</span></div>'
+            '</div>',
+            balance_before,
+            obj.balance_after
         )
     
     @display(description="Payment")
-    def payment_display(self, obj):
-        """Display related payment."""
-        if obj.payment:
-            return format_html(
-                '<a href="{}" style="color: #007bff;">#{}</a><br><small>{}</small>',
-                reverse('admin:django_cfg_payments_universalpayment_change', args=[obj.payment.id]),
-                obj.payment.internal_payment_id[:8],
-                obj.payment.get_status_display()
-            )
-        return "—"
-    
-    @display(description="Subscription")
-    def subscription_display(self, obj):
-        """Display related subscription."""
-        if obj.subscription:
+    def payment_link_display(self, obj):
+        """Display payment link if available."""
+        if obj.payment_id:
             return format_html(
-                '<a href="{}" style="color: #007bff;">{}</a><br><small>{}</small>',
-                reverse('admin:django_cfg_payments_subscription_change', args=[obj.subscription.id]),
-                obj.subscription.endpoint_group.display_name,
-                obj.subscription.get_tier_display()
+                '<a href="/admin/payments/universalpayment/{}/change/" '
+                'class="text-blue-600 hover:text-blue-800 dark:text-blue-400 dark:hover:text-blue-200">'
+                '🔗 Payment'
+                '</a>',
+                obj.payment_id
             )
-        return "—"
+        return format_html('<span class="text-gray-500">—</span>')
     
-    @display(description="Created")
+    @display(description="Created", ordering='created_at')
     def created_at_display(self, obj):
-        """Display creation date."""
-        return naturaltime(obj.created_at)
-    
-    def transaction_details(self, obj):
-        """Show detailed transaction information."""
+        """Display creation timestamp."""
         return format_html(
-            '<div style="line-height: 1.6;">'
-            '<strong>Transaction Details:</strong><br>'
-            '• ID: {}<br>'
-            '• User: {} ({})<br>'
-            '• Type: {}<br>'
-            '• Amount: <span style="color: {};">${:.2f}</span><br>'
-            '• Description: {}<br>'
-            '• Created: {}<br>'
-            '{}'
-            '{}'
+            '<div class="text-xs">'
+            '<div>{}</div>'
+            '<div class="text-gray-500">{}</div>'
             '</div>',
-            obj.id,
-            obj.user.get_full_name() or 'No name',
-            obj.user.email,
-            obj.get_transaction_type_display(),
-            '#28a745' if obj.amount_usd > 0 else '#dc3545',
-            obj.amount_usd,
-            obj.description,
             obj.created_at.strftime('%Y-%m-%d %H:%M:%S'),
-            f'• Payment: {obj.payment.internal_payment_id}<br>' if obj.payment else '',
-            f'• Subscription: {obj.subscription.endpoint_group.name}<br>' if obj.subscription else ''
+            naturaltime(obj.created_at)
         )
-    
-    transaction_details.short_description = "Transaction Details"
-    
-    def related_objects(self, obj):
-        """Show related objects."""
-        html = '<div style="line-height: 1.6;">'
-        
-        if obj.payment:
-            html += f'''
-            <strong>Related Payment:</strong><br>
-            • ID: {obj.payment.internal_payment_id}<br>
-            • Status: {obj.payment.get_status_display()}<br>
-            • Amount: ${obj.payment.amount_usd:.2f}<br>
-            • Provider: {obj.payment.provider}<br>
-            '''
-        
-        if obj.subscription:
-            html += f'''
-            <strong>Related Subscription:</strong><br>
-            • Endpoint: {obj.subscription.endpoint_group.display_name}<br>
-            • Tier: {obj.subscription.get_tier_display()}<br>
-            • Status: {obj.subscription.get_status_display()}<br>
-            • Usage: {obj.subscription.usage_current}/{obj.subscription.usage_limit}<br>
-            '''
-        
-        if obj.metadata:
-            html += '<strong>Metadata:</strong><br>'
-            for key, value in obj.metadata.items():
-                html += f'• {key}: {value}<br>'
-        
-        html += '</div>'
-        return format_html(html)
-    
-    related_objects.short_description = "Related Objects"