feat: initial commit

.github/workflows/release.yml

@@ -12,7 +12,7 @@ on:
         required: true
         type: string
       ref:
-        description: 'Git ref to checkout'
+        description: 'Git ref to checkout (branch, tag, or SHA)'
         required: true
         default: 'main'
         type: string
@@ -29,57 +29,73 @@ jobs:
     runs-on: ubuntu-latest
     permissions:
       contents: read
-      packages: write # Needed if you switch to GHCR, good practice
 
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
         with:
-          ref: ${{ inputs.ref || github.ref }}
+          ref: ${{ github.event.inputs.ref }}
+        if: github.event_name == 'workflow_dispatch'
+
+      - name: Checkout code (non-manual)
+        uses: actions/checkout@v4
+        if: github.event_name != 'workflow_dispatch'
 
       - name: Log in to Docker Hub
-        uses: docker/login-action@v3
+        uses: docker/login-action@65b78e6e13532edd9afa3aa52ac7964289d1a9c1
         with:
           username: ${{ secrets.DOCKERHUB_USERNAME }}
           password: ${{ secrets.DOCKERHUB_TOKEN }}
 
-      # This action handles all the logic for tags (nightly vs release vs custom)
-      - name: Docker Metadata
-        id: meta
-        uses: docker/metadata-action@v5
-        with:
-          images: ${{ secrets.DOCKERHUB_USERNAME }}/${{ env.IMAGE_NAME }}
-          tags: |
-            # Logic for Push to Main -> nightly
-            type=raw,value=nightly,enable=${{ github.event_name == 'push' }}
-            # Logic for Release -> semver and latest
-            type=semver,pattern={{version}},enable=${{ github.event_name == 'release' }}
-            type=raw,value=latest,enable=${{ github.event_name == 'release' }}
-            # Logic for Manual Dispatch -> custom input
-            type=raw,value=${{ inputs.tag }},enable=${{ github.event_name == 'workflow_dispatch' }}
-
       - name: Set up QEMU
         uses: docker/setup-qemu-action@v3
 
       - name: Set up Docker Buildx
         uses: docker/setup-buildx-action@v3
 
-      - name: Build and push
+      - name: Build and push nightly image
+        if: github.event_name == 'push'
         uses: docker/build-push-action@v6
         with:
           context: .
           file: ./docker/prod/django/Dockerfile
           push: true
           provenance: false
-          # Pass the calculated tags from the meta step
-          tags: ${{ steps.meta.outputs.tags }}
-          labels: ${{ steps.meta.outputs.labels }}
           build-args: |
-            VERSION=${{ steps.meta.outputs.version }}
+            VERSION=nightly
+          tags: ${{ secrets.DOCKERHUB_USERNAME }}/${{ env.IMAGE_NAME }}:nightly
           platforms: linux/amd64,linux/arm64
+          cache-from: type=gha
+          cache-to: type=gha,mode=max
 
-          # --- CACHE CONFIGURATION ---
-          # We set a specific 'scope' key.
-          # This allows the Release tag to see the cache created by the Main branch.
-          cache-from: type=gha,scope=build-cache
-          cache-to: type=gha,mode=max,scope=build-cache
+      - name: Build and push release image
+        if: github.event_name == 'release'
+        uses: docker/build-push-action@v6
+        with:
+          context: .
+          file: ./docker/prod/django/Dockerfile
+          push: true
+          provenance: false
+          build-args: |
+            VERSION=${{ github.event.release.tag_name }}
+          tags: |
+            ${{ secrets.DOCKERHUB_USERNAME }}/${{ env.IMAGE_NAME }}:latest
+            ${{ secrets.DOCKERHUB_USERNAME }}/${{ env.IMAGE_NAME }}:${{ github.event.release.tag_name }}
+          platforms: linux/amd64,linux/arm64
+          cache-from: type=gha
+          cache-to: type=gha,mode=max
+
+      - name: Build and push custom image
+        if: github.event_name == 'workflow_dispatch'
+        uses: docker/build-push-action@v6
+        with:
+          context: .
+          file: ./docker/prod/django/Dockerfile
+          push: true
+          provenance: false
+          build-args: |
+            VERSION=${{ github.event.inputs.tag }}
+          tags: ${{ secrets.DOCKERHUB_USERNAME }}/${{ env.IMAGE_NAME }}:${{ github.event.inputs.tag }}
+          platforms: linux/amd64,linux/arm64
+          cache-from: type=gha
+          cache-to: type=gha,mode=max

.gitignore

@@ -123,7 +123,6 @@ celerybeat.pid
 
 # Environments
 .env
-.prod.env
 .venv
 env/
 venv/
@@ -161,7 +160,3 @@ cython_debug/
 #  and can be added to the global gitignore or merged into this file.  For a more nuclear
 #  option (not recommended) you can uncomment the following to ignore the entire idea folder.
 .idea/
-
-node_modules/
-postgres_data/
-.prod.env

.vscode/settings.json

@@ -1,8 +0,0 @@
-{
-    "djlint.showInstallError": false,
-    "files.associations": {
-        "*.css": "tailwindcss"
-    },
-    "tailwindCSS.experimental.configFile": "frontend/src/styles/tailwind.css",
-    "djlint.profile": "django",
-}

README.md

@@ -13,7 +13,6 @@
   <a href="#key-features">Features</a> •
   <a href="#how-to-use">Usage</a> •
   <a href="#how-it-works">How</a> •
-  <a href="#mcp-server">MCP Server</a> •
   <a href="#help-us-translate-wygiwyh">Translate</a> •
   <a href="#caveats-and-warnings">Caveats and Warnings</a> •
   <a href="#built-with">Built with</a>
@@ -127,7 +126,6 @@ To create the first user, open the container's console using Unraid's UI, by cli
 
 | variable                      | type        | default                           | explanation                                                                                                                                                                                                                              |
 |-------------------------------|-------------|-----------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
-| INTERNAL_PORT                 | int         | 8000                              | The port on which the app listens on. Defaults to 8000 if not set.                                                                                                                                                                       |
 | DJANGO_ALLOWED_HOSTS          | string      | localhost 127.0.0.1               | A list of space separated domains and IPs representing the host/domain names that WYGIWYH site can serve. [Click here](https://docs.djangoproject.com/en/5.1/ref/settings/#allowed-hosts) for more details                               |
 | HTTPS_ENABLED                 | true\|false | false                             | Whether to use secure cookies. If this is set to true, the cookie will be marked as “secure”, which means browsers may ensure that the cookie is only sent under an HTTPS connection                                                     |
 | URL                           | string      | http://localhost http://127.0.0.1 | A list of space separated domains and IPs (with the protocol) representing the trusted origins for unsafe requests (e.g. POST). [Click here](https://docs.djangoproject.com/en/5.1/ref/settings/#csrf-trusted-origins ) for more details |
@@ -145,10 +143,7 @@ To create the first user, open the container's console using Unraid's UI, by cli
 | DEMO                          | true\|false | false                             | If demo mode is enabled.                                                                                                                                                                                                                 |
 | ADMIN_EMAIL                   | string      | None                              | Automatically creates an admin account with this email. Must have `ADMIN_PASSWORD` also set.                                                                                                                                             |
 | ADMIN_PASSWORD                | string      | None                              | Automatically creates an admin account with this password. Must have `ADMIN_EMAIL` also set.                                                                                                                                             |
-| CHECK_FOR_UPDATES             | true\|false | true                              | Check and notify users about new versions. The check is done by doing a single query to Github's API every 12 hours.                                                                                                                     |
-| DJANGO_VITE_DEV_MODE          | true\|false | false                             | Enables Vite dev server mode for frontend development. When true, assets are served from Vite's dev server instead of the build manifest. For development only!                                                                          |
-| DJANGO_VITE_DEV_SERVER_PORT   | int         | 5173                              | The port where Vite's dev server is running. Only used when DJANGO_VITE_DEV_MODE is true. For development only!                                                                                                                          |
-| DJANGO_VITE_DEV_SERVER_HOST   | string      | localhost                         | The host where Vite's dev server is running. Only used when DJANGO_VITE_DEV_MODE is true. For development only!                                                                                                                          |
+| CHECK_FOR_UPDATES             | bool        | true                              | Check and notify users about new versions. The check is done by doing a single query to Github's API every 12 hours.                                                                                  |
 
 ## OIDC Configuration
 
@@ -187,10 +182,6 @@ Check out our [Wiki](https://github.com/eitchtee/WYGIWYH/wiki) for more informat
 > [!NOTE]
 > Login with your github account
 
-# MCP Server
-
-[IZIme07](https://github.com/IZIme07) has kindly created an MCP Server for WYGIWYH that you can self-host. [Check it out at MCP-WYGIWYH](https://github.com/ReNewator/MCP-WYGIWYH)!
-
 # Caveats and Warnings
 
 - I'm not an accountant, some terms and even calculations might be wrong. Make sure to open an issue if you see anything that could be improved.

app/WYGIWYH/settings.py

@@ -11,7 +11,6 @@ https://docs.djangoproject.com/en/5.1/ref/settings/
 """
 
 import os
-import re
 import sys
 from pathlib import Path
 
@@ -47,7 +46,7 @@ INSTALLED_APPS = [
     "django.contrib.sites",
     "whitenoise.runserver_nostatic",
     "django.contrib.staticfiles",
-    "django_vite",
+    "webpack_boilerplate",
     "django.contrib.humanize",
     "django.contrib.postgres",
     "django_browser_reload",
@@ -129,23 +128,12 @@ STORAGES = {
 
 WHITENOISE_MANIFEST_STRICT = False
 
-
-def immutable_file_test(path, url):
-    # Match vite (rollup)-generated hashes, à la, `some_file-CSliV9zW.js`
-    return re.match(r"^.+[.-][0-9a-zA-Z_-]{8,12}\..+$", url)
-
-
-WHITENOISE_IMMUTABLE_FILE_TEST = immutable_file_test
-
 WSGI_APPLICATION = "WYGIWYH.wsgi.application"
 
 
 # Database
 # https://docs.djangoproject.com/en/5.1/ref/settings/#databases
 
-THREADS = int(os.getenv("GUNICORN_THREADS", 1))
-MAX_POOL_SIZE = THREADS + 1
-
 DATABASES = {
     "default": {
         "ENGINE": "django.db.backends.postgresql",
@@ -154,17 +142,6 @@ DATABASES = {
         "PASSWORD": os.getenv("SQL_PASSWORD", "password"),
         "HOST": os.getenv("SQL_HOST", "localhost"),
         "PORT": os.getenv("SQL_PORT", "5432"),
-        "CONN_MAX_AGE": 0,
-        "CONN_HEALTH_CHECKS": True,
-        "OPTIONS": {
-            "pool": {
-                "min_size": 1,
-                "max_size": MAX_POOL_SIZE,
-                "timeout": 10,
-                "max_lifetime": 600,
-                "max_idle": 300,
-            },
-        },
     }
 }
 
@@ -312,7 +289,7 @@ STATIC_URL = "static/"
 STATIC_ROOT = BASE_DIR / "static_files"
 
 STATICFILES_DIRS = [
-    ROOT_DIR / "frontend" / "build",
+    ROOT_DIR / "frontend/build",
     BASE_DIR / "static",
 ]
 
@@ -328,11 +305,9 @@ CACHES = {
     }
 }
 
-DJANGO_VITE_ASSETS_PATH = STATIC_ROOT
-DJANGO_VITE_MANIFEST_PATH = DJANGO_VITE_ASSETS_PATH / "manifest.json"
-DJANGO_VITE_DEV_MODE = os.getenv("DJANGO_VITE_DEV_MODE", "false").lower() == "true"
-DJANGO_VITE_DEV_SERVER_PORT = int(os.getenv("DJANGO_VITE_DEV_SERVER_PORT", "5173"))
-DJANGO_VITE_DEV_SERVER_HOST = os.getenv("DJANGO_VITE_DEV_SERVER_HOST", "localhost")
+WEBPACK_LOADER = {
+    "MANIFEST_FILE": ROOT_DIR / "frontend/build/manifest.json",
+}
 
 # Default primary key field type
 # https://docs.djangoproject.com/en/5.1/ref/settings/#default-auto-field
@@ -379,11 +354,8 @@ ACCOUNT_ADAPTER = "allauth.account.adapter.DefaultAccountAdapter"
 SOCIALACCOUNT_ADAPTER = "allauth.socialaccount.adapter.DefaultSocialAccountAdapter"
 
 # CRISPY FORMS
-CRISPY_ALLOWED_TEMPLATE_PACKS = [
-    "crispy_forms/pure_text",
-    "crispy-daisyui",
-]
-CRISPY_TEMPLATE_PACK = "crispy-daisyui"
+CRISPY_ALLOWED_TEMPLATE_PACKS = ["bootstrap5", "crispy_forms/pure_text"]
+CRISPY_TEMPLATE_PACK = "bootstrap5"
 
 SESSION_EXPIRE_AT_BROWSER_CLOSE = False
 SESSION_COOKIE_AGE = int(os.getenv("SESSION_EXPIRY_TIME", 2678400))  # 31 days
@@ -407,7 +379,7 @@ DEBUG_TOOLBAR_PANELS = [
     "debug_toolbar.panels.signals.SignalsPanel",
     "debug_toolbar.panels.redirects.RedirectsPanel",
     "debug_toolbar.panels.profiling.ProfilingPanel",
-    # "cachalot.panels.CachalotPanel",
+    "cachalot.panels.CachalotPanel",
 ]
 INTERNAL_IPS = [
     "127.0.0.1",

app/apps/accounts/forms.py

@@ -1,21 +1,21 @@
-from apps.accounts.models import Account, AccountGroup
-from apps.common.fields.forms.dynamic_select import (
-    DynamicModelChoiceField,
-    DynamicModelMultipleChoiceField,
-)
-from apps.common.widgets.crispy.daisyui import Switch
-from apps.common.widgets.crispy.submit import NoClassSubmit
-from apps.common.widgets.decimal import ArbitraryDecimalDisplayNumberInput
-from apps.common.widgets.tom_select import TomSelect
-from apps.currencies.models import Currency
-from apps.transactions.models import TransactionCategory, TransactionTag
+from crispy_bootstrap5.bootstrap5 import Switch
 from crispy_forms.bootstrap import FormActions
 from crispy_forms.helper import FormHelper
-from crispy_forms.layout import Column, Field, Layout, Row
+from crispy_forms.layout import Layout, Field, Column, Row
 from django import forms
-from django.db.models import Q
 from django.utils.translation import gettext_lazy as _
 
+from apps.accounts.models import Account
+from apps.accounts.models import AccountGroup
+from apps.common.fields.forms.dynamic_select import (
+    DynamicModelMultipleChoiceField,
+    DynamicModelChoiceField,
+)
+from apps.common.widgets.crispy.submit import NoClassSubmit
+from apps.common.widgets.tom_select import TomSelect
+from apps.transactions.models import TransactionCategory, TransactionTag
+from apps.common.widgets.decimal import ArbitraryDecimalDisplayNumberInput
+
 
 class AccountGroupForm(forms.ModelForm):
     class Meta:
@@ -36,13 +36,17 @@ class AccountGroupForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -75,18 +79,6 @@ class AccountForm(forms.ModelForm):
 
         self.fields["group"].queryset = AccountGroup.objects.all()
 
-        if self.instance.id:
-            qs = Currency.objects.filter(
-                Q(is_archived=False) | Q(accounts=self.instance.id)
-            ).distinct()
-            self.fields["currency"].queryset = qs
-            self.fields["exchange_currency"].queryset = qs
-
-        else:
-            qs = Currency.objects.filter(Q(is_archived=False))
-            self.fields["currency"].queryset = qs
-            self.fields["exchange_currency"].queryset = qs
-
         self.helper = FormHelper()
         self.helper.form_tag = False
         self.helper.form_method = "post"
@@ -102,13 +94,17 @@ class AccountForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -146,8 +142,9 @@ class AccountBalanceForm(forms.Form):
         self.helper.layout = Layout(
             "new_balance",
             Row(
-                Column("category"),
-                Column("tags"),
+                Column("category", css_class="form-group col-md-6 mb-0"),
+                Column("tags", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             Field("account_id"),
         )

app/apps/accounts/migrations/0016_account_untracked_by.py

@@ -1,20 +0,0 @@
-# Generated by Django 5.2.4 on 2025-08-09 05:52
-
-from django.conf import settings
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('accounts', '0015_alter_account_owner_alter_account_shared_with_and_more'),
-        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='account',
-            name='untracked_by',
-            field=models.ManyToManyField(blank=True, related_name='untracked_accounts', to=settings.AUTH_USER_MODEL),
-        ),
-    ]

app/apps/accounts/models.py

@@ -1,11 +1,11 @@
 from django.conf import settings
 from django.core.exceptions import ValidationError
 from django.db import models
+from django.db.models import Q
 from django.utils.translation import gettext_lazy as _
 
-from apps.common.middleware.thread_local import get_current_user
-from apps.common.models import SharedObject, SharedObjectManager
 from apps.transactions.models import Transaction
+from apps.common.models import SharedObject, SharedObjectManager
 
 
 class AccountGroup(SharedObject):
@@ -62,11 +62,6 @@ class Account(SharedObject):
         verbose_name=_("Archived"),
         help_text=_("Archived accounts don't show up nor count towards your net worth"),
     )
-    untracked_by = models.ManyToManyField(
-        settings.AUTH_USER_MODEL,
-        blank=True,
-        related_name="untracked_accounts",
-    )
 
     objects = SharedObjectManager()
     all_objects = models.Manager()  # Unfiltered manager
@@ -80,10 +75,6 @@ class Account(SharedObject):
     def __str__(self):
         return self.name
 
-    def is_untracked_by(self):
-        user = get_current_user()
-        return self.untracked_by.filter(pk=user.pk).exists()
-
     def clean(self):
         super().clean()
         if self.exchange_currency == self.currency:

app/apps/accounts/services.py

@@ -1,33 +0,0 @@
-from decimal import Decimal
-
-from django.db import models
-
-from apps.accounts.models import Account
-from apps.transactions.models import Transaction
-
-
-def get_account_balance(account: Account, paid_only: bool = True) -> Decimal:
-    """
-    Calculate account balance (income - expense).
-
-    Args:
-        account: Account instance to calculate balance for.
-        paid_only: If True, only count paid transactions (current balance).
-                   If False, count all transactions (projected balance).
-
-    Returns:
-        Decimal: The calculated balance (income - expense).
-    """
-    filters = {"account": account}
-    if paid_only:
-        filters["is_paid"] = True
-
-    income = Transaction.objects.filter(
-        type=Transaction.Type.INCOME, **filters
-    ).aggregate(total=models.Sum("amount"))["total"] or Decimal("0")
-
-    expense = Transaction.objects.filter(
-        type=Transaction.Type.EXPENSE, **filters
-    ).aggregate(total=models.Sum("amount"))["total"] or Decimal("0")
-
-    return income - expense

app/apps/accounts/tests.py

@@ -1,5 +1,3 @@
-from datetime import date
-
 from django.test import TestCase
 
 from apps.accounts.models import Account, AccountGroup
@@ -41,135 +39,3 @@ class AccountTests(TestCase):
             exchange_currency=self.exchange_currency,
         )
         self.assertEqual(account.exchange_currency, self.exchange_currency)
-
-
-class GetAccountBalanceServiceTests(TestCase):
-    """Tests for the get_account_balance service function"""
-
-    def setUp(self):
-        """Set up test data"""
-        from apps.transactions.models import Transaction
-        self.Transaction = Transaction
-        
-        self.currency = Currency.objects.create(
-            code="BRL", name="Brazilian Real", decimal_places=2, prefix="R$ "
-        )
-        self.account_group = AccountGroup.objects.create(name="Service Test Group")
-        self.account = Account.objects.create(
-            name="Service Test Account", group=self.account_group, currency=self.currency
-        )
-
-    def test_balance_with_no_transactions(self):
-        """Test balance is 0 when no transactions exist"""
-        from apps.accounts.services import get_account_balance
-        from decimal import Decimal
-        
-        balance = get_account_balance(self.account, paid_only=True)
-        self.assertEqual(balance, Decimal("0"))
-
-    def test_current_balance_only_counts_paid(self):
-        """Test current balance only counts paid transactions"""
-        from apps.accounts.services import get_account_balance
-        from decimal import Decimal
-        
-        # Paid income
-        self.Transaction.objects.create(
-            account=self.account,
-            type=self.Transaction.Type.INCOME,
-            amount=Decimal("100.00"),
-            is_paid=True,
-            date=date(2025, 1, 1),
-            description="Paid income",
-        )
-        # Unpaid income (should not count)
-        self.Transaction.objects.create(
-            account=self.account,
-            type=self.Transaction.Type.INCOME,
-            amount=Decimal("50.00"),
-            is_paid=False,
-            date=date(2025, 1, 1),
-            description="Unpaid income",
-        )
-        # Paid expense
-        self.Transaction.objects.create(
-            account=self.account,
-            type=self.Transaction.Type.EXPENSE,
-            amount=Decimal("30.00"),
-            is_paid=True,
-            date=date(2025, 1, 1),
-            description="Paid expense",
-        )
-
-        balance = get_account_balance(self.account, paid_only=True)
-        self.assertEqual(balance, Decimal("70.00"))  # 100 - 30
-
-    def test_projected_balance_counts_all(self):
-        """Test projected balance counts all transactions"""
-        from apps.accounts.services import get_account_balance
-        from decimal import Decimal
-        
-        # Paid income
-        self.Transaction.objects.create(
-            account=self.account,
-            type=self.Transaction.Type.INCOME,
-            amount=Decimal("100.00"),
-            is_paid=True,
-            date=date(2025, 1, 1),
-            description="Paid income",
-        )
-        # Unpaid income
-        self.Transaction.objects.create(
-            account=self.account,
-            type=self.Transaction.Type.INCOME,
-            amount=Decimal("50.00"),
-            is_paid=False,
-            date=date(2025, 1, 1),
-            description="Unpaid income",
-        )
-        # Paid expense
-        self.Transaction.objects.create(
-            account=self.account,
-            type=self.Transaction.Type.EXPENSE,
-            amount=Decimal("30.00"),
-            is_paid=True,
-            date=date(2025, 1, 1),
-            description="Paid expense",
-        )
-        # Unpaid expense
-        self.Transaction.objects.create(
-            account=self.account,
-            type=self.Transaction.Type.EXPENSE,
-            amount=Decimal("20.00"),
-            is_paid=False,
-            date=date(2025, 1, 1),
-            description="Unpaid expense",
-        )
-
-        balance = get_account_balance(self.account, paid_only=False)
-        self.assertEqual(balance, Decimal("100.00"))  # (100 + 50) - (30 + 20)
-
-    def test_balance_defaults_to_paid_only(self):
-        """Test that paid_only defaults to True"""
-        from apps.accounts.services import get_account_balance
-        from decimal import Decimal
-        
-        self.Transaction.objects.create(
-            account=self.account,
-            type=self.Transaction.Type.INCOME,
-            amount=Decimal("100.00"),
-            is_paid=True,
-            date=date(2025, 1, 1),
-            description="Paid",
-        )
-        self.Transaction.objects.create(
-            account=self.account,
-            type=self.Transaction.Type.INCOME,
-            amount=Decimal("50.00"),
-            is_paid=False,
-            date=date(2025, 1, 1),
-            description="Unpaid",
-        )
-
-        balance = get_account_balance(self.account)  # defaults to paid_only=True
-        self.assertEqual(balance, Decimal("100.00"))
-

app/apps/accounts/urls.py

@@ -31,11 +31,6 @@ urlpatterns = [
         views.account_take_ownership,
         name="account_take_ownership",
     ),
-    path(
-        "account/<int:pk>/toggle-untracked/",
-        views.account_toggle_untracked,
-        name="account_toggle_untracked",
-    ),
     path("account-groups/", views.account_groups_index, name="account_groups_index"),
     path("account-groups/list/", views.account_groups_list, name="account_groups_list"),
     path("account-groups/add/", views.account_group_add, name="account_group_add"),

app/apps/accounts/views/accounts.py

@@ -155,26 +155,6 @@ def account_delete(request, pk):
     )
 
 
-@only_htmx
-@login_required
-@require_http_methods(["GET"])
-def account_toggle_untracked(request, pk):
-    account = get_object_or_404(Account, id=pk)
-    if account.is_untracked_by():
-        account.untracked_by.remove(request.user)
-        messages.success(request, _("Account is now tracked"))
-    else:
-        account.untracked_by.add(request.user)
-        messages.success(request, _("Account is now untracked"))
-
-    return HttpResponse(
-        status=204,
-        headers={
-            "HX-Trigger": "updated",
-        },
-    )
-
-
 @only_htmx
 @login_required
 @require_http_methods(["GET"])

app/apps/accounts/views/balance.py

@@ -11,13 +11,23 @@ from django.utils.translation import gettext_lazy as _
 
 from apps.accounts.forms import AccountBalanceFormSet
 from apps.accounts.models import Account, Transaction
-from apps.accounts.services import get_account_balance
 from apps.common.decorators.htmx import only_htmx
 
 
 @only_htmx
 @login_required
 def account_reconciliation(request):
+    def get_account_balance(account):
+        income = Transaction.objects.filter(
+            account=account, type=Transaction.Type.INCOME, is_paid=True
+        ).aggregate(total=models.Sum("amount"))["total"] or Decimal("0")
+
+        expense = Transaction.objects.filter(
+            account=account, type=Transaction.Type.EXPENSE, is_paid=True
+        ).aggregate(total=models.Sum("amount"))["total"] or Decimal("0")
+
+        return income - expense
+
     initial_data = [
         {
             "account_id": account.id,

app/apps/api/fields/transactions.py

@@ -10,19 +10,15 @@ from apps.transactions.models import (
 
 @extend_schema_field(
     {
-        "oneOf": [{"type": "string"}, {"type": "integer"}, {"type": "null"}],
-        "description": "TransactionCategory ID or name. If the name doesn't exist, a new one will be created. Can be null if no category is assigned.",
+        "oneOf": [{"type": "string"}, {"type": "integer"}],
+        "description": "TransactionCategory ID or name. If the name doesn't exist, a new one will be created",
     }
 )
 class TransactionCategoryField(serializers.Field):
     def to_representation(self, value):
-        if value is None:
-            return None
         return {"id": value.id, "name": value.name}
 
     def to_internal_value(self, data):
-        if data is None:
-            return None
         if isinstance(data, int):
             try:
                 return TransactionCategory.objects.get(pk=data)

app/apps/api/serializers/__init__.py

@@ -2,5 +2,3 @@ from .transactions import *
 from .accounts import *
 from .currencies import *
 from .dca import *
-from .imports import *
-

app/apps/api/serializers/accounts.py

@@ -67,12 +67,3 @@ class AccountSerializer(serializers.ModelSerializer):
             setattr(instance, attr, value)
         instance.save()
         return instance
-
-
-class AccountBalanceSerializer(serializers.Serializer):
-    """Serializer for account balance response."""
-
-    current_balance = serializers.DecimalField(max_digits=20, decimal_places=10)
-    projected_balance = serializers.DecimalField(max_digits=20, decimal_places=10)
-    currency = CurrencySerializer()
-

app/apps/api/serializers/imports.py

@@ -1,41 +0,0 @@
-from rest_framework import serializers
-
-from apps.import_app.models import ImportProfile, ImportRun
-
-
-class ImportProfileSerializer(serializers.ModelSerializer):
-    """Serializer for listing import profiles."""
-
-    class Meta:
-        model = ImportProfile
-        fields = ["id", "name", "version", "yaml_config"]
-
-
-class ImportRunSerializer(serializers.ModelSerializer):
-    """Serializer for listing import runs."""
-
-    class Meta:
-        model = ImportRun
-        fields = [
-            "id",
-            "status",
-            "profile",
-            "file_name",
-            "logs",
-            "processed_rows",
-            "total_rows",
-            "successful_rows",
-            "skipped_rows",
-            "failed_rows",
-            "started_at",
-            "finished_at",
-        ]
-
-
-class ImportFileSerializer(serializers.Serializer):
-    """Serializer for uploading a file to import using an existing profile."""
-
-    profile_id = serializers.PrimaryKeyRelatedField(
-        queryset=ImportProfile.objects.all(), source="profile"
-    )
-    file = serializers.FileField()

app/apps/api/serializers/transactions.py

@@ -138,7 +138,6 @@ class RecurringTransactionSerializer(serializers.ModelSerializer):
     def update(self, instance, validated_data):
         instance = super().update(instance, validated_data)
         instance.update_unpaid_transactions()
-        instance.generate_upcoming_transactions()
         return instance
 
 

app/apps/api/tests/__init__.py

@@ -1,4 +0,0 @@
-# Import all test classes for Django test discovery
-from .test_imports import *
-from .test_accounts import *
-

app/apps/api/tests/test_accounts.py

@@ -1,99 +0,0 @@
-from datetime import date
-from decimal import Decimal
-
-from django.contrib.auth import get_user_model
-from django.test import TestCase, override_settings
-from rest_framework import status
-from rest_framework.test import APIClient
-
-from apps.accounts.models import Account, AccountGroup
-from apps.currencies.models import Currency
-from apps.transactions.models import Transaction
-
-
-@override_settings(
-    STORAGES={
-        "default": {"BACKEND": "django.core.files.storage.FileSystemStorage"},
-        "staticfiles": {
-            "BACKEND": "django.contrib.staticfiles.storage.StaticFilesStorage"
-        },
-    },
-    WHITENOISE_AUTOREFRESH=True,
-)
-class AccountBalanceAPITests(TestCase):
-    """Tests for the Account Balance API endpoint"""
-
-    def setUp(self):
-        """Set up test data"""
-        User = get_user_model()
-        self.user = User.objects.create_user(
-            email="testuser@test.com", password="testpass123"
-        )
-        self.client = APIClient()
-        self.client.force_authenticate(user=self.user)
-
-        self.currency = Currency.objects.create(
-            code="USD", name="US Dollar", decimal_places=2, prefix="$ "
-        )
-        self.account_group = AccountGroup.objects.create(name="Test Group")
-        self.account = Account.objects.create(
-            name="Test Account", group=self.account_group, currency=self.currency
-        )
-
-        # Create some transactions
-        Transaction.objects.create(
-            account=self.account,
-            type=Transaction.Type.INCOME,
-            amount=Decimal("500.00"),
-            is_paid=True,
-            date=date(2025, 1, 1),
-            description="Paid income",
-        )
-        Transaction.objects.create(
-            account=self.account,
-            type=Transaction.Type.INCOME,
-            amount=Decimal("200.00"),
-            is_paid=False,
-            date=date(2025, 1, 15),
-            description="Unpaid income",
-        )
-        Transaction.objects.create(
-            account=self.account,
-            type=Transaction.Type.EXPENSE,
-            amount=Decimal("100.00"),
-            is_paid=True,
-            date=date(2025, 1, 10),
-            description="Paid expense",
-        )
-
-    def test_get_balance_success(self):
-        """Test successful balance retrieval"""
-        response = self.client.get(f"/api/accounts/{self.account.id}/balance/")
-
-        self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertIn("current_balance", response.data)
-        self.assertIn("projected_balance", response.data)
-        self.assertIn("currency", response.data)
-
-        # Current: 500 - 100 = 400
-        self.assertEqual(Decimal(response.data["current_balance"]), Decimal("400.00"))
-        # Projected: (500 + 200) - 100 = 600
-        self.assertEqual(Decimal(response.data["projected_balance"]), Decimal("600.00"))
-
-        # Check currency data
-        self.assertEqual(response.data["currency"]["code"], "USD")
-
-    def test_get_balance_nonexistent_account(self):
-        """Test balance for non-existent account returns 404"""
-        response = self.client.get("/api/accounts/99999/balance/")
-
-        self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
-
-    def test_get_balance_unauthenticated(self):
-        """Test unauthenticated request returns 403"""
-        unauthenticated_client = APIClient()
-        response = unauthenticated_client.get(
-            f"/api/accounts/{self.account.id}/balance/"
-        )
-
-        self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)

app/apps/api/tests/test_imports.py

@@ -1,404 +0,0 @@
-from io import BytesIO
-from unittest.mock import patch
-
-from django.contrib.auth import get_user_model
-from django.core.files.uploadedfile import SimpleUploadedFile
-from django.test import TestCase, override_settings
-from rest_framework import status
-from rest_framework.test import APIClient
-
-from apps.import_app.models import ImportProfile, ImportRun
-
-
-@override_settings(
-    STORAGES={
-        "default": {"BACKEND": "django.core.files.storage.FileSystemStorage"},
-        "staticfiles": {
-            "BACKEND": "django.contrib.staticfiles.storage.StaticFilesStorage"
-        },
-    },
-    WHITENOISE_AUTOREFRESH=True,
-)
-class ImportAPITests(TestCase):
-    """Tests for the Import API endpoint"""
-
-    def setUp(self):
-        """Set up test data"""
-        User = get_user_model()
-        self.user = User.objects.create_user(
-            email="testuser@test.com", password="testpass123"
-        )
-        self.client = APIClient()
-        self.client.force_authenticate(user=self.user)
-
-        # Create a basic import profile with minimal valid YAML config
-        self.profile = ImportProfile.objects.create(
-            name="Test Profile",
-            version=ImportProfile.Versions.VERSION_1,
-            yaml_config="""
-file_type: csv
-date_format: "%Y-%m-%d"
-column_mapping:
-  date:
-    source: date
-  description:
-    source: description
-  amount:
-    source: amount
-  transaction_type:
-    detection_method: always_expense
-  is_paid:
-    detection_method: always_paid
-  account:
-    source: account
-    match_field: name
-""",
-        )
-
-    @patch("apps.import_app.tasks.process_import.defer")
-    @patch("django.core.files.storage.FileSystemStorage.save")
-    @patch("django.core.files.storage.FileSystemStorage.path")
-    def test_create_import_success(self, mock_path, mock_save, mock_defer):
-        """Test successful file upload creates ImportRun and queues task"""
-        mock_save.return_value = "test_file.csv"
-        mock_path.return_value = "/usr/src/app/temp/test_file.csv"
-
-        csv_content = b"date,description,amount,account\n2025-01-01,Test,100,Main"
-        file = SimpleUploadedFile(
-            "test_file.csv", csv_content, content_type="text/csv"
-        )
-
-        response = self.client.post(
-            "/api/import/import/",
-            {"profile_id": self.profile.id, "file": file},
-            format="multipart",
-        )
-
-        self.assertEqual(response.status_code, status.HTTP_202_ACCEPTED)
-        self.assertIn("import_run_id", response.data)
-        self.assertEqual(response.data["status"], "queued")
-
-        # Verify ImportRun was created
-        import_run = ImportRun.objects.get(id=response.data["import_run_id"])
-        self.assertEqual(import_run.profile, self.profile)
-        self.assertEqual(import_run.file_name, "test_file.csv")
-
-        # Verify task was deferred
-        mock_defer.assert_called_once_with(
-            import_run_id=import_run.id,
-            file_path="/usr/src/app/temp/test_file.csv",
-            user_id=self.user.id,
-        )
-
-    def test_create_import_missing_profile(self):
-        """Test request without profile_id returns 400"""
-        csv_content = b"date,description,amount\n2025-01-01,Test,100"
-        file = SimpleUploadedFile(
-            "test_file.csv", csv_content, content_type="text/csv"
-        )
-
-        response = self.client.post(
-            "/api/import/import/",
-            {"file": file},
-            format="multipart",
-        )
-
-        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
-        self.assertIn("profile_id", response.data)
-
-    def test_create_import_missing_file(self):
-        """Test request without file returns 400"""
-        response = self.client.post(
-            "/api/import/import/",
-            {"profile_id": self.profile.id},
-            format="multipart",
-        )
-
-        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
-        self.assertIn("file", response.data)
-
-    def test_create_import_invalid_profile(self):
-        """Test request with non-existent profile returns 400"""
-        csv_content = b"date,description,amount\n2025-01-01,Test,100"
-        file = SimpleUploadedFile(
-            "test_file.csv", csv_content, content_type="text/csv"
-        )
-
-        response = self.client.post(
-            "/api/import/import/",
-            {"profile_id": 99999, "file": file},
-            format="multipart",
-        )
-
-        self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
-        self.assertIn("profile_id", response.data)
-
-    @patch("apps.import_app.tasks.process_import.defer")
-    @patch("django.core.files.storage.FileSystemStorage.save")
-    @patch("django.core.files.storage.FileSystemStorage.path")
-    def test_create_import_xlsx(self, mock_path, mock_save, mock_defer):
-        """Test successful XLSX file upload"""
-        mock_save.return_value = "test_file.xlsx"
-        mock_path.return_value = "/usr/src/app/temp/test_file.xlsx"
-
-        # Create a simple XLSX-like content (just for the upload test)
-        xlsx_content = BytesIO(b"PK\x03\x04")  # XLSX files start with PK header
-        file = SimpleUploadedFile(
-            "test_file.xlsx",
-            xlsx_content.getvalue(),
-            content_type="application/vnd.openxmlformats-officedocument.spreadsheetml.sheet",
-        )
-
-        response = self.client.post(
-            "/api/import/import/",
-            {"profile_id": self.profile.id, "file": file},
-            format="multipart",
-        )
-
-        self.assertEqual(response.status_code, status.HTTP_202_ACCEPTED)
-        self.assertIn("import_run_id", response.data)
-
-    def test_unauthenticated_request(self):
-        """Test unauthenticated request returns 403"""
-        unauthenticated_client = APIClient()
-
-        csv_content = b"date,description,amount\n2025-01-01,Test,100"
-        file = SimpleUploadedFile(
-            "test_file.csv", csv_content, content_type="text/csv"
-        )
-
-        response = unauthenticated_client.post(
-            "/api/import/import/",
-            {"profile_id": self.profile.id, "file": file},
-            format="multipart",
-        )
-
-        self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
-
-
-@override_settings(
-    STORAGES={
-        "default": {"BACKEND": "django.core.files.storage.FileSystemStorage"},
-        "staticfiles": {
-            "BACKEND": "django.contrib.staticfiles.storage.StaticFilesStorage"
-        },
-    },
-    WHITENOISE_AUTOREFRESH=True,
-)
-class ImportProfileAPITests(TestCase):
-    """Tests for the Import Profile API endpoints"""
-
-    def setUp(self):
-        """Set up test data"""
-        User = get_user_model()
-        self.user = User.objects.create_user(
-            email="testuser@test.com", password="testpass123"
-        )
-        self.client = APIClient()
-        self.client.force_authenticate(user=self.user)
-
-        self.profile1 = ImportProfile.objects.create(
-            name="Profile 1",
-            version=ImportProfile.Versions.VERSION_1,
-            yaml_config="""
-file_type: csv
-date_format: "%Y-%m-%d"
-column_mapping:
-  date:
-    source: date
-  description:
-    source: description
-  amount:
-    source: amount
-  transaction_type:
-    detection_method: always_expense
-  is_paid:
-    detection_method: always_paid
-  account:
-    source: account
-    match_field: name
-""",
-        )
-        self.profile2 = ImportProfile.objects.create(
-            name="Profile 2",
-            version=ImportProfile.Versions.VERSION_1,
-            yaml_config="""
-file_type: csv
-date_format: "%Y-%m-%d"
-column_mapping:
-  date:
-    source: date
-  description:
-    source: description
-  amount:
-    source: amount
-  transaction_type:
-    detection_method: always_income
-  is_paid:
-    detection_method: always_unpaid
-  account:
-    source: account
-    match_field: name
-""",
-        )
-
-    def test_list_profiles(self):
-        """Test listing all profiles"""
-        response = self.client.get("/api/import/profiles/")
-
-        self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(response.data["count"], 2)
-        self.assertEqual(len(response.data["results"]), 2)
-
-    def test_retrieve_profile(self):
-        """Test retrieving a specific profile"""
-        response = self.client.get(f"/api/import/profiles/{self.profile1.id}/")
-
-        self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(response.data["id"], self.profile1.id)
-        self.assertEqual(response.data["name"], "Profile 1")
-        self.assertIn("yaml_config", response.data)
-
-    def test_retrieve_nonexistent_profile(self):
-        """Test retrieving a non-existent profile returns 404"""
-        response = self.client.get("/api/import/profiles/99999/")
-
-        self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
-
-    def test_profiles_unauthenticated(self):
-        """Test unauthenticated request returns 403"""
-        unauthenticated_client = APIClient()
-        response = unauthenticated_client.get("/api/import/profiles/")
-
-        self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
-
-
-@override_settings(
-    STORAGES={
-        "default": {"BACKEND": "django.core.files.storage.FileSystemStorage"},
-        "staticfiles": {
-            "BACKEND": "django.contrib.staticfiles.storage.StaticFilesStorage"
-        },
-    },
-    WHITENOISE_AUTOREFRESH=True,
-)
-class ImportRunAPITests(TestCase):
-    """Tests for the Import Run API endpoints"""
-
-    def setUp(self):
-        """Set up test data"""
-        User = get_user_model()
-        self.user = User.objects.create_user(
-            email="testuser@test.com", password="testpass123"
-        )
-        self.client = APIClient()
-        self.client.force_authenticate(user=self.user)
-
-        self.profile1 = ImportProfile.objects.create(
-            name="Profile 1",
-            version=ImportProfile.Versions.VERSION_1,
-            yaml_config="""
-file_type: csv
-date_format: "%Y-%m-%d"
-column_mapping:
-  date:
-    source: date
-  description:
-    source: description
-  amount:
-    source: amount
-  transaction_type:
-    detection_method: always_expense
-  is_paid:
-    detection_method: always_paid
-  account:
-    source: account
-    match_field: name
-""",
-        )
-        self.profile2 = ImportProfile.objects.create(
-            name="Profile 2",
-            version=ImportProfile.Versions.VERSION_1,
-            yaml_config="""
-file_type: csv
-date_format: "%Y-%m-%d"
-column_mapping:
-  date:
-    source: date
-  description:
-    source: description
-  amount:
-    source: amount
-  transaction_type:
-    detection_method: always_income
-  is_paid:
-    detection_method: always_unpaid
-  account:
-    source: account
-    match_field: name
-""",
-        )
-
-        # Create import runs
-        self.run1 = ImportRun.objects.create(
-            profile=self.profile1,
-            file_name="file1.csv",
-            status=ImportRun.Status.FINISHED,
-        )
-        self.run2 = ImportRun.objects.create(
-            profile=self.profile1,
-            file_name="file2.csv",
-            status=ImportRun.Status.QUEUED,
-        )
-        self.run3 = ImportRun.objects.create(
-            profile=self.profile2,
-            file_name="file3.csv",
-            status=ImportRun.Status.FINISHED,
-        )
-
-    def test_list_all_runs(self):
-        """Test listing all runs"""
-        response = self.client.get("/api/import/runs/")
-
-        self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(response.data["count"], 3)
-        self.assertEqual(len(response.data["results"]), 3)
-
-    def test_list_runs_by_profile(self):
-        """Test filtering runs by profile_id"""
-        response = self.client.get(f"/api/import/runs/?profile_id={self.profile1.id}")
-
-        self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(response.data["count"], 2)
-        for run in response.data["results"]:
-            self.assertEqual(run["profile"], self.profile1.id)
-
-    def test_list_runs_by_other_profile(self):
-        """Test filtering runs by another profile_id"""
-        response = self.client.get(f"/api/import/runs/?profile_id={self.profile2.id}")
-
-        self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(response.data["count"], 1)
-        self.assertEqual(response.data["results"][0]["profile"], self.profile2.id)
-
-    def test_retrieve_run(self):
-        """Test retrieving a specific run"""
-        response = self.client.get(f"/api/import/runs/{self.run1.id}/")
-
-        self.assertEqual(response.status_code, status.HTTP_200_OK)
-        self.assertEqual(response.data["id"], self.run1.id)
-        self.assertEqual(response.data["file_name"], "file1.csv")
-        self.assertEqual(response.data["status"], "FINISHED")
-
-    def test_retrieve_nonexistent_run(self):
-        """Test retrieving a non-existent run returns 404"""
-        response = self.client.get("/api/import/runs/99999/")
-
-        self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
-
-    def test_runs_unauthenticated(self):
-        """Test unauthenticated request returns 403"""
-        unauthenticated_client = APIClient()
-        response = unauthenticated_client.get("/api/import/runs/")
-
-        self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)

app/apps/api/urls.py

@@ -16,11 +16,7 @@ router.register(r"currencies", views.CurrencyViewSet)
 router.register(r"exchange-rates", views.ExchangeRateViewSet)
 router.register(r"dca/strategies", views.DCAStrategyViewSet)
 router.register(r"dca/entries", views.DCAEntryViewSet)
-router.register(r"import/profiles", views.ImportProfileViewSet, basename="import-profiles")
-router.register(r"import/runs", views.ImportRunViewSet, basename="import-runs")
-router.register(r"import/import", views.ImportViewSet, basename="import-import")
 
 urlpatterns = [
     path("", include(router.urls)),
 ]
-

app/apps/api/views/__init__.py

@@ -2,5 +2,3 @@ from .transactions import *
 from .accounts import *
 from .currencies import *
 from .dca import *
-from .imports import *
-

app/apps/api/views/accounts.py

@@ -1,18 +1,11 @@
-from drf_spectacular.utils import extend_schema, extend_schema_view
 from rest_framework import viewsets
-from rest_framework.decorators import action
-from rest_framework.permissions import IsAuthenticated
-from rest_framework.response import Response
 
-from apps.accounts.models import AccountGroup, Account
-from apps.accounts.services import get_account_balance
 from apps.api.custom.pagination import CustomPageNumberPagination
-from apps.api.serializers import AccountGroupSerializer, AccountSerializer, AccountBalanceSerializer
+from apps.accounts.models import AccountGroup, Account
+from apps.api.serializers import AccountGroupSerializer, AccountSerializer
 
 
 class AccountGroupViewSet(viewsets.ModelViewSet):
-    """ViewSet for managing account groups."""
-
     queryset = AccountGroup.objects.all()
     serializer_class = AccountGroupSerializer
     pagination_class = CustomPageNumberPagination
@@ -21,16 +14,7 @@ class AccountGroupViewSet(viewsets.ModelViewSet):
         return AccountGroup.objects.all().order_by("id")
 
 
-@extend_schema_view(
-    balance=extend_schema(
-        summary="Get account balance",
-        description="Returns the current and projected balance for the account, along with currency data.",
-        responses={200: AccountBalanceSerializer},
-    ),
-)
 class AccountViewSet(viewsets.ModelViewSet):
-    """ViewSet for managing accounts."""
-
     queryset = Account.objects.all()
     serializer_class = AccountSerializer
     pagination_class = CustomPageNumberPagination
@@ -41,20 +25,3 @@ class AccountViewSet(viewsets.ModelViewSet):
             .order_by("id")
             .select_related("group", "currency", "exchange_currency")
         )
-
-    @action(detail=True, methods=["get"], permission_classes=[IsAuthenticated])
-    def balance(self, request, pk=None):
-        """Get current and projected balance for an account."""
-        account = self.get_object()
-        
-        current_balance = get_account_balance(account, paid_only=True)
-        projected_balance = get_account_balance(account, paid_only=False)
-        
-        serializer = AccountBalanceSerializer({
-            "current_balance": current_balance,
-            "projected_balance": projected_balance,
-            "currency": account.currency,
-        })
-        
-        return Response(serializer.data)
-

app/apps/api/views/imports.py

@@ -1,123 +0,0 @@
-from django.core.files.storage import FileSystemStorage
-from drf_spectacular.types import OpenApiTypes
-from drf_spectacular.utils import OpenApiParameter, extend_schema, extend_schema_view, inline_serializer
-from rest_framework import serializers as drf_serializers
-from rest_framework import status, viewsets
-from rest_framework.parsers import MultiPartParser
-from rest_framework.permissions import IsAuthenticated
-from rest_framework.response import Response
-
-from apps.api.serializers import ImportFileSerializer, ImportProfileSerializer, ImportRunSerializer
-from apps.import_app.models import ImportProfile, ImportRun
-from apps.import_app.tasks import process_import
-
-
-@extend_schema_view(
-    list=extend_schema(
-        summary="List import profiles",
-        description="Returns a paginated list of all available import profiles.",
-    ),
-    retrieve=extend_schema(
-        summary="Get import profile",
-        description="Returns the details of a specific import profile by ID.",
-    ),
-)
-class ImportProfileViewSet(viewsets.ReadOnlyModelViewSet):
-    """ViewSet for listing and retrieving import profiles."""
-
-    queryset = ImportProfile.objects.all()
-    serializer_class = ImportProfileSerializer
-    permission_classes = [IsAuthenticated]
-
-
-@extend_schema_view(
-    list=extend_schema(
-        summary="List import runs",
-        description="Returns a paginated list of import runs. Optionally filter by profile_id.",
-        parameters=[
-            OpenApiParameter(
-                name="profile_id",
-                type=int,
-                location=OpenApiParameter.QUERY,
-                description="Filter runs by profile ID",
-                required=False,
-            ),
-        ],
-    ),
-    retrieve=extend_schema(
-        summary="Get import run",
-        description="Returns the details of a specific import run by ID, including status and logs.",
-    ),
-)
-class ImportRunViewSet(viewsets.ReadOnlyModelViewSet):
-    """ViewSet for listing and retrieving import runs."""
-
-    queryset = ImportRun.objects.all().order_by("-id")
-    serializer_class = ImportRunSerializer
-    permission_classes = [IsAuthenticated]
-
-    def get_queryset(self):
-        queryset = super().get_queryset()
-        profile_id = self.request.query_params.get("profile_id")
-        if profile_id:
-            queryset = queryset.filter(profile_id=profile_id)
-        return queryset
-
-
-@extend_schema_view(
-    create=extend_schema(
-        summary="Import file",
-        description="Upload a CSV or XLSX file to import using an existing import profile. The import is queued and processed asynchronously.",
-        request={
-            "multipart/form-data": {
-                "type": "object",
-                "properties": {
-                    "profile_id": {"type": "integer", "description": "ID of the ImportProfile to use"},
-                    "file": {"type": "string", "format": "binary", "description": "CSV or XLSX file to import"},
-                },
-                "required": ["profile_id", "file"],
-            },
-        },
-        responses={
-            202: inline_serializer(
-                name="ImportResponse",
-                fields={
-                    "import_run_id": drf_serializers.IntegerField(),
-                    "status": drf_serializers.CharField(),
-                },
-            ),
-        },
-    ),
-)
-class ImportViewSet(viewsets.ViewSet):
-    """ViewSet for importing data via file upload."""
-
-    permission_classes = [IsAuthenticated]
-    parser_classes = [MultiPartParser]
-
-    def create(self, request):
-        serializer = ImportFileSerializer(data=request.data)
-        serializer.is_valid(raise_exception=True)
-
-        profile = serializer.validated_data["profile"]
-        uploaded_file = serializer.validated_data["file"]
-
-        # Save file to temp location
-        fs = FileSystemStorage(location="/usr/src/app/temp")
-        filename = fs.save(uploaded_file.name, uploaded_file)
-        file_path = fs.path(filename)
-
-        # Create ImportRun record
-        import_run = ImportRun.objects.create(profile=profile, file_name=filename)
-
-        # Queue import task
-        process_import.defer(
-            import_run_id=import_run.id,
-            file_path=file_path,
-            user_id=request.user.id,
-        )
-
-        return Response(
-            {"import_run_id": import_run.id, "status": "queued"},
-            status=status.HTTP_202_ACCEPTED,
-        )

app/apps/api/views/transactions.py

@@ -1,5 +1,3 @@
-from copy import deepcopy
-
 from rest_framework import viewsets
 
 from apps.api.custom.pagination import CustomPageNumberPagination
@@ -32,9 +30,8 @@ class TransactionViewSet(viewsets.ModelViewSet):
         transaction_created.send(sender=instance)
 
     def perform_update(self, serializer):
-        old_data = deepcopy(self.get_object())
         instance = serializer.save()
-        transaction_updated.send(sender=instance, old_data=old_data)
+        transaction_updated.send(sender=instance)
 
     def partial_update(self, request, *args, **kwargs):
         kwargs["partial"] = True

app/apps/common/fields/forms/dynamic_select.py

@@ -139,6 +139,7 @@ class DynamicModelMultipleChoiceField(forms.ModelMultipleChoiceField):
                     instance.save()
                     return instance
         except Exception as e:
+            print(e)
             raise ValidationError(_("Error creating new instance"))
 
     def clean(self, value):

app/apps/common/forms.py

@@ -1,13 +1,14 @@
-from apps.common.models import SharedObject
-from apps.common.widgets.crispy.submit import NoClassSubmit
-from apps.common.widgets.tom_select import TomSelect, TomSelectMultiple
 from crispy_forms.bootstrap import FormActions
-from crispy_forms.helper import FormHelper
-from crispy_forms.layout import HTML, Div, Field, Layout, Submit
 from django import forms
 from django.contrib.auth import get_user_model
-from django.core.exceptions import ValidationError
 from django.utils.translation import gettext_lazy as _
+from django.core.exceptions import ValidationError
+from crispy_forms.helper import FormHelper
+from crispy_forms.layout import Layout, Field, Submit, Div, HTML
+
+from apps.common.widgets.tom_select import TomSelect, TomSelectMultiple
+from apps.common.models import SharedObject
+from apps.common.widgets.crispy.submit import NoClassSubmit
 
 User = get_user_model()
 
@@ -38,7 +39,6 @@ class SharedObjectForm(forms.Form):
         choices=SharedObject.Visibility.choices,
         required=True,
         label=_("Visibility"),
-        widget=TomSelect(clear_button=False),
         help_text=_(
             "Private: Only shown for the owner and shared users. Only editable by the owner."
             "<br/>"
@@ -48,6 +48,9 @@ class SharedObjectForm(forms.Form):
 
     class Meta:
         fields = ["visibility", "shared_with_users"]
+        widgets = {
+            "visibility": TomSelect(clear_button=False),
+        }
 
     def __init__(self, *args, **kwargs):
         # Get the current user to filter available sharing options
@@ -70,10 +73,12 @@ class SharedObjectForm(forms.Form):
         self.helper.layout = Layout(
             Field("owner"),
             Field("visibility"),
-            HTML('<hr class="hr my-3">'),
+            HTML("<hr>"),
             Field("shared_with_users"),
             FormActions(
-                NoClassSubmit("submit", _("Save"), css_class="btn btn-primary"),
+                NoClassSubmit(
+                    "submit", _("Save"), css_class="btn btn-outline-primary w-100"
+                ),
             ),
         )
 

app/apps/common/functions/decimals.py

@@ -9,8 +9,5 @@ def truncate_decimal(value, decimal_places):
     :param decimal_places: The number of decimal places to keep
     :return: Truncated Decimal value
     """
-    if isinstance(value, (int, float)):
-        value = Decimal(str(value))
-
     multiplier = Decimal(10**decimal_places)
     return (value * multiplier).to_integral_value(rounding=ROUND_DOWN) / multiplier

app/apps/common/functions/format.py

@@ -5,12 +5,7 @@ from django.utils.formats import get_format as original_get_format
 def get_format(format_type=None, lang=None, use_l10n=None):
     user = get_current_user()
 
-    if (
-        user
-        and user.is_authenticated
-        and hasattr(user, "settings")
-        and use_l10n is not False
-    ):
+    if user and user.is_authenticated and hasattr(user, "settings") and use_l10n:
         user_settings = user.settings
         if format_type == "THOUSAND_SEPARATOR":
             number_format = getattr(user_settings, "number_format", None)
@@ -18,13 +13,11 @@ def get_format(format_type=None, lang=None, use_l10n=None):
                 return "."
             elif number_format == "CD":
                 return ","
-            elif number_format == "SD" or number_format == "SC":
-                return " "
         elif format_type == "DECIMAL_SEPARATOR":
             number_format = getattr(user_settings, "number_format", None)
-            if number_format == "DC" or number_format == "SC":
+            if number_format == "DC":
                 return ","
-            elif number_format == "CD" or number_format == "SD":
+            elif number_format == "CD":
                 return "."
         elif format_type == "SHORT_DATE_FORMAT":
             date_format = getattr(user_settings, "date_format", None)

app/apps/common/tasks.py

@@ -17,18 +17,13 @@ logger = logging.getLogger(__name__)
 
 
 @app.periodic(cron="0 4 * * *")
-@app.task(
-    lock="remove_old_jobs",
-    queueing_lock="remove_old_jobs",
-    pass_context=True,
-    name="remove_old_jobs",
-)
+@app.task(queueing_lock="remove_old_jobs", pass_context=True, name="remove_old_jobs")
 async def remove_old_jobs(context, timestamp):
     try:
         return await builtin_tasks.remove_old_jobs(
             context,
             max_hours=744,
-            remove_failed=True,
+            remove_error=True,
             remove_cancelled=True,
             remove_aborted=True,
         )
@@ -41,11 +36,7 @@ async def remove_old_jobs(context, timestamp):
 
 
 @app.periodic(cron="0 6 1 * *")
-@app.task(
-    lock="remove_expired_sessions",
-    queueing_lock="remove_expired_sessions",
-    name="remove_expired_sessions",
-)
+@app.task(queueing_lock="remove_expired_sessions", name="remove_expired_sessions")
 async def remove_expired_sessions(timestamp=None):
     """Cleanup expired sessions by using Django management command."""
     try:
@@ -58,7 +49,7 @@ async def remove_expired_sessions(timestamp=None):
 
 
 @app.periodic(cron="0 8 * * *")
-@app.task(lock="reset_demo_data", name="reset_demo_data")
+@app.task(name="reset_demo_data")
 def reset_demo_data(timestamp=None):
     """
     Wipes the database and loads fresh demo data if DEMO mode is active.
@@ -95,7 +86,9 @@ def reset_demo_data(timestamp=None):
 
 
 @app.periodic(cron="0 */12 * * *")  # Every 12 hours
-@app.task(lock="check_for_updates", name="check_for_updates")
+@app.task(
+    name="check_for_updates",
+)
 def check_for_updates(timestamp=None):
     if not settings.CHECK_FOR_UPDATES:
         return "CHECK_FOR_UPDATES is disabled"

app/apps/common/templatetags/crispy_extra.py

@@ -1,13 +0,0 @@
-from django import forms, template
-
-register = template.Library()
-
-
-@register.filter
-def is_input(field):
-    return isinstance(field.field.widget, forms.TextInput)
-
-
-@register.filter
-def is_textarea(field):
-    return isinstance(field.field.widget, forms.Textarea)

app/apps/common/templatetags/toast_bg.py

@@ -11,7 +11,7 @@ def toast_bg(tags):
     elif "warning" in tags:
         return "warning"
     elif "error" in tags:
-        return "error"
+        return "danger"
     elif "info" in tags:
         return "info"
 

app/apps/common/views.py

@@ -91,12 +91,6 @@ def month_year_picker(request):
         for date in all_months
     ]
 
-    today_url = (
-        reverse(url, kwargs={"month": current_date.month, "year": current_date.year})
-        if url
-        else ""
-    )
-
     return render(
         request,
         "common/fragments/month_year_picker.html",
@@ -104,7 +98,6 @@ def month_year_picker(request):
             "month_year_data": result,
             "current_month": current_month,
             "current_year": current_year,
-            "today_url": today_url,
         },
     )
 

app/apps/common/widgets/crispy/daisyui.py

@@ -1,5 +0,0 @@
-from crispy_forms.layout import Field
-
-
-class Switch(Field):
-    template = "crispy-daisyui/layout/switch.html"

app/apps/common/widgets/datepicker.py

@@ -1,14 +1,15 @@
 import datetime
 
-from apps.common.functions.format import get_format
+from django.forms import widgets
+from django.utils import formats, translation, dates
+from django.utils.translation import gettext_lazy as _
+
 from apps.common.utils.django import (
+    django_to_python_datetime,
     django_to_airdatepicker_datetime,
     django_to_airdatepicker_datetime_separated,
-    django_to_python_datetime,
 )
-from django.forms import widgets
-from django.utils import dates, formats, translation
-from django.utils.translation import gettext_lazy as _
+from apps.common.functions.format import get_format
 
 
 class AirDatePickerInput(widgets.DateInput):
@@ -51,8 +52,6 @@ class AirDatePickerInput(widgets.DateInput):
     def build_attrs(self, base_attrs, extra_attrs=None):
         attrs = super().build_attrs(base_attrs, extra_attrs)
 
-        attrs["class"] = attrs.get("class", "") + " input"
-
         attrs["data-now-button-txt"] = _("Today")
         attrs["data-auto-close"] = str(self.auto_close).lower()
         attrs["data-clear-button"] = str(self.clear_button).lower()

app/apps/common/widgets/decimal.py

@@ -35,8 +35,9 @@ class ArbitraryDecimalDisplayNumberInput(forms.TextInput):
         self.attrs.update(
             {
                 "x-data": "",
-                "x-mask:dynamic": f"$money($input, '{get_format('DECIMAL_SEPARATOR')}', '{get_format('THOUSAND_SEPARATOR')}', '30')",
-                "x-on:keyup": "if (!['Control', 'Shift', 'Alt', 'Meta'].includes($event.key) && !(($event.ctrlKey || $event.metaKey) && $event.key.toLowerCase() === 'a')) $el.dispatchEvent(new Event('input'))",
+                "x-mask:dynamic": f"$money($input, '{get_format('DECIMAL_SEPARATOR')}', "
+                f"'{get_format('THOUSAND_SEPARATOR')}', '30')",
+                "x-on:keyup": "$el.dispatchEvent(new Event('input'))",
             }
         )
 

app/apps/common/widgets/tom_select.py

@@ -1,4 +1,4 @@
-from django.forms import SelectMultiple, widgets
+from django.forms import widgets, SelectMultiple
 from django.urls import reverse
 from django.utils.translation import gettext_lazy as _
 
@@ -17,7 +17,7 @@ class TomSelect(widgets.Select):
         checkboxes=False,
         group_by=None,
         *args,
-        **kwargs,
+        **kwargs
     ):
         super().__init__(attrs, *args, **kwargs)
         self.remove_button = remove_button

app/apps/currencies/exchange_rates/fetcher.py

@@ -4,7 +4,13 @@ from datetime import timedelta
 from django.db.models import QuerySet
 from django.utils import timezone
 
-import apps.currencies.exchange_rates.providers as providers
+from apps.currencies.exchange_rates.providers import (
+    SynthFinanceProvider,
+    SynthFinanceStockProvider,
+    CoinGeckoFreeProvider,
+    CoinGeckoProProvider,
+    TransitiveRateProvider,
+)
 from apps.currencies.models import ExchangeRateService, ExchangeRate, Currency
 
 logger = logging.getLogger(__name__)
@@ -12,12 +18,11 @@ logger = logging.getLogger(__name__)
 
 # Map service types to provider classes
 PROVIDER_MAPPING = {
-    "coingecko_free": providers.CoinGeckoFreeProvider,
-    "coingecko_pro": providers.CoinGeckoProProvider,
-    "transitive": providers.TransitiveRateProvider,
-    "frankfurter": providers.FrankfurterProvider,
-    "twelvedata": providers.TwelveDataProvider,
-    "twelvedatamarkets": providers.TwelveDataMarketsProvider,
+    "synth_finance": SynthFinanceProvider,
+    "synth_finance_stock": SynthFinanceStockProvider,
+    "coingecko_free": CoinGeckoFreeProvider,
+    "coingecko_pro": CoinGeckoProProvider,
+    "transitive": TransitiveRateProvider,
 }
 
 
@@ -198,63 +203,21 @@ class ExchangeRateFetcher:
 
                 if provider.rates_inverted:
                     # If rates are inverted, we need to swap currencies
-                    if service.singleton:
-                        # Try to get the last automatically created exchange rate
-                        exchange_rate = (
-                            ExchangeRate.objects.filter(
-                                automatic=True,
-                                from_currency=to_currency,
-                                to_currency=from_currency,
-                            )
-                            .order_by("-date")
-                            .first()
-                        )
-                    else:
-                        exchange_rate = None
-
-                    if not exchange_rate:
-                        ExchangeRate.objects.create(
-                            automatic=True,
-                            from_currency=to_currency,
-                            to_currency=from_currency,
-                            rate=rate,
-                            date=timezone.now(),
-                        )
-                    else:
-                        exchange_rate.rate = rate
-                        exchange_rate.date = timezone.now()
-                        exchange_rate.save()
-
+                    ExchangeRate.objects.create(
+                        from_currency=to_currency,
+                        to_currency=from_currency,
+                        rate=rate,
+                        date=timezone.now(),
+                    )
                     processed_pairs.add((to_currency.id, from_currency.id))
                 else:
                     # If rates are not inverted, we can use them as is
-                    if service.singleton:
-                        # Try to get the last automatically created exchange rate
-                        exchange_rate = (
-                            ExchangeRate.objects.filter(
-                                automatic=True,
-                                from_currency=from_currency,
-                                to_currency=to_currency,
-                            )
-                            .order_by("-date")
-                            .first()
-                        )
-                    else:
-                        exchange_rate = None
-
-                    if not exchange_rate:
-                        ExchangeRate.objects.create(
-                            automatic=True,
-                            from_currency=from_currency,
-                            to_currency=to_currency,
-                            rate=rate,
-                            date=timezone.now(),
-                        )
-                    else:
-                        exchange_rate.rate = rate
-                        exchange_rate.date = timezone.now()
-                        exchange_rate.save()
-
+                    ExchangeRate.objects.create(
+                        from_currency=from_currency,
+                        to_currency=to_currency,
+                        rate=rate,
+                        date=timezone.now(),
+                    )
                     processed_pairs.add((from_currency.id, to_currency.id))
 
             service.last_fetch = timezone.now()

app/apps/currencies/exchange_rates/providers.py

@@ -13,6 +13,70 @@ from apps.currencies.exchange_rates.base import ExchangeRateProvider
 logger = logging.getLogger(__name__)
 
 
+class SynthFinanceProvider(ExchangeRateProvider):
+    """Implementation for Synth Finance API (synthfinance.com)"""
+
+    BASE_URL = "https://api.synthfinance.com/rates/live"
+    rates_inverted = False  # SynthFinance returns non-inverted rates
+
+    def __init__(self, api_key: str = None):
+        super().__init__(api_key)
+        self.session = requests.Session()
+        self.session.headers.update({"Authorization": f"Bearer {self.api_key}"})
+
+    def get_rates(
+        self, target_currencies: QuerySet, exchange_currencies: set
+    ) -> List[Tuple[Currency, Currency, Decimal]]:
+        results = []
+        currency_groups = {}
+        for currency in target_currencies:
+            if currency.exchange_currency in exchange_currencies:
+                group = currency_groups.setdefault(currency.exchange_currency.code, [])
+                group.append(currency)
+
+        for base_currency, currencies in currency_groups.items():
+            try:
+                to_currencies = ",".join(
+                    currency.code
+                    for currency in currencies
+                    if currency.code != base_currency
+                )
+                response = self.session.get(
+                    f"{self.BASE_URL}",
+                    params={"from": base_currency, "to": to_currencies},
+                )
+                response.raise_for_status()
+                data = response.json()
+                rates = data["data"]["rates"]
+
+                for currency in currencies:
+                    if currency.code == base_currency:
+                        rate = Decimal("1")
+                    else:
+                        rate = Decimal(str(rates[currency.code]))
+                    # Return the rate as is, without inversion
+                    results.append((currency.exchange_currency, currency, rate))
+
+                credits_used = data["meta"]["credits_used"]
+                credits_remaining = data["meta"]["credits_remaining"]
+                logger.info(
+                    f"Synth Finance API call: {credits_used} credits used, {credits_remaining} remaining"
+                )
+            except requests.RequestException as e:
+                logger.error(
+                    f"Error fetching rates from Synth Finance API for base {base_currency}: {e}"
+                )
+            except KeyError as e:
+                logger.error(
+                    f"Unexpected response structure from Synth Finance API for base {base_currency}: {e}"
+                )
+            except Exception as e:
+                logger.error(
+                    f"Unexpected error processing Synth Finance data for base {base_currency}: {e}"
+                )
+        return results
+
+
 class CoinGeckoFreeProvider(ExchangeRateProvider):
     """Implementation for CoinGecko Free API"""
 
@@ -88,6 +152,71 @@ class CoinGeckoProProvider(CoinGeckoFreeProvider):
         self.session.headers.update({"x-cg-pro-api-key": api_key})
 
 
+class SynthFinanceStockProvider(ExchangeRateProvider):
+    """Implementation for Synth Finance API Real-Time Prices endpoint (synthfinance.com)"""
+
+    BASE_URL = "https://api.synthfinance.com/tickers"
+    rates_inverted = True
+
+    def __init__(self, api_key: str = None):
+        super().__init__(api_key)
+        self.session = requests.Session()
+        self.session.headers.update(
+            {"Authorization": f"Bearer {self.api_key}", "accept": "application/json"}
+        )
+
+    def get_rates(
+        self, target_currencies: QuerySet, exchange_currencies: set
+    ) -> List[Tuple[Currency, Currency, Decimal]]:
+        results = []
+
+        for currency in target_currencies:
+            if currency.exchange_currency not in exchange_currencies:
+                continue
+
+            try:
+                # Same currency has rate of 1
+                if currency.code == currency.exchange_currency.code:
+                    rate = Decimal("1")
+                    results.append((currency.exchange_currency, currency, rate))
+                    continue
+
+                # Fetch real-time price for this ticker
+                response = self.session.get(
+                    f"{self.BASE_URL}/{currency.code}/real-time"
+                )
+                response.raise_for_status()
+                data = response.json()
+
+                # Use fair market value as the rate
+                rate = Decimal(data["data"]["fair_market_value"])
+                results.append((currency.exchange_currency, currency, rate))
+
+                # Log API usage
+                credits_used = data["meta"]["credits_used"]
+                credits_remaining = data["meta"]["credits_remaining"]
+                logger.info(
+                    f"Synth Finance API call for {currency.code}: {credits_used} credits used, {credits_remaining} remaining"
+                )
+            except requests.RequestException as e:
+                logger.error(
+                    f"Error fetching rate from Synth Finance API for ticker {currency.code}: {e}",
+                    exc_info=True,
+                )
+            except KeyError as e:
+                logger.error(
+                    f"Unexpected response structure from Synth Finance API for ticker {currency.code}: {e}",
+                    exc_info=True,
+                )
+            except Exception as e:
+                logger.error(
+                    f"Unexpected error processing Synth Finance data for ticker {currency.code}: {e}",
+                    exc_info=True,
+                )
+
+        return results
+
+
 class TransitiveRateProvider(ExchangeRateProvider):
     """Calculates exchange rates through paths of existing rates"""
 
@@ -177,329 +306,3 @@ class TransitiveRateProvider(ExchangeRateProvider):
                     queue.append((neighbor, path + [neighbor], current_rate * rate))
 
         return None, None
-
-
-class FrankfurterProvider(ExchangeRateProvider):
-    """Implementation for the Frankfurter API (frankfurter.dev)"""
-
-    BASE_URL = "https://api.frankfurter.dev/v1/latest"
-    rates_inverted = (
-        False  # Frankfurter returns non-inverted rates (e.g., 1 EUR = 1.1 USD)
-    )
-
-    def __init__(self, api_key: str = None):
-        """
-        Initializes the provider. The Frankfurter API does not require an API key,
-        so the api_key parameter is ignored.
-        """
-        super().__init__(api_key)
-        self.session = requests.Session()
-
-    @classmethod
-    def requires_api_key(cls) -> bool:
-        return False
-
-    def get_rates(
-        self, target_currencies: QuerySet, exchange_currencies: set
-    ) -> List[Tuple[Currency, Currency, Decimal]]:
-        results = []
-        currency_groups = {}
-        # Group target currencies by their exchange (base) currency to minimize API calls
-        for currency in target_currencies:
-            if currency.exchange_currency in exchange_currencies:
-                group = currency_groups.setdefault(currency.exchange_currency.code, [])
-                group.append(currency)
-
-        # Make one API call for each base currency
-        for base_currency, currencies in currency_groups.items():
-            try:
-                # Create a comma-separated list of target currency codes
-                to_currencies = ",".join(
-                    currency.code
-                    for currency in currencies
-                    if currency.code != base_currency
-                )
-
-                # If there are no target currencies other than the base, skip the API call
-                if not to_currencies:
-                    # Handle the case where the only request is for the base rate (e.g., USD to USD)
-                    for currency in currencies:
-                        if currency.code == base_currency:
-                            results.append(
-                                (currency.exchange_currency, currency, Decimal("1"))
-                            )
-                    continue
-
-                response = self.session.get(
-                    self.BASE_URL,
-                    params={"base": base_currency, "symbols": to_currencies},
-                )
-                response.raise_for_status()
-                data = response.json()
-                rates = data["rates"]
-
-                # Process the returned rates
-                for currency in currencies:
-                    if currency.code == base_currency:
-                        # The rate for the base currency to itself is always 1
-                        rate = Decimal("1")
-                    else:
-                        rate = Decimal(str(rates[currency.code]))
-
-                    results.append((currency.exchange_currency, currency, rate))
-
-            except requests.RequestException as e:
-                logger.error(
-                    f"Error fetching rates from Frankfurter API for base {base_currency}: {e}"
-                )
-            except KeyError as e:
-                logger.error(
-                    f"Unexpected response structure from Frankfurter API for base {base_currency}: {e}"
-                )
-            except Exception as e:
-                logger.error(
-                    f"Unexpected error processing Frankfurter data for base {base_currency}: {e}"
-                )
-        return results
-
-
-class TwelveDataProvider(ExchangeRateProvider):
-    """Implementation for the Twelve Data API (twelvedata.com)"""
-
-    BASE_URL = "https://api.twelvedata.com/exchange_rate"
-    rates_inverted = (
-        False  # The API returns direct rates, e.g., for EUR/USD it's 1 EUR = X USD
-    )
-
-    def __init__(self, api_key: str):
-        """
-        Initializes the provider with an API key and a requests session.
-        """
-        super().__init__(api_key)
-        self.session = requests.Session()
-
-    @classmethod
-    def requires_api_key(cls) -> bool:
-        """This provider requires an API key."""
-        return True
-
-    def get_rates(
-        self, target_currencies: QuerySet, exchange_currencies: set
-    ) -> List[Tuple[Currency, Currency, Decimal]]:
-        """
-        Fetches exchange rates from the Twelve Data API for the given currency pairs.
-
-        This provider makes one API call for each requested currency pair.
-        """
-        results = []
-
-        for target_currency in target_currencies:
-            # Ensure the target currency's exchange currency is one we're interested in
-            if target_currency.exchange_currency not in exchange_currencies:
-                continue
-
-            base_currency = target_currency.exchange_currency
-
-            # The exchange rate for the same currency is always 1
-            if base_currency.code == target_currency.code:
-                rate = Decimal("1")
-                results.append((base_currency, target_currency, rate))
-                continue
-
-            # Construct the symbol in the format "BASE/TARGET", e.g., "EUR/USD"
-            symbol = f"{base_currency.code}/{target_currency.code}"
-
-            try:
-                params = {
-                    "symbol": symbol,
-                    "apikey": self.api_key,
-                }
-
-                response = self.session.get(self.BASE_URL, params=params)
-                response.raise_for_status()  # Raise an HTTPError for bad responses (4xx or 5xx)
-
-                data = response.json()
-
-                # The API may return an error message in a JSON object
-                if "rate" not in data:
-                    error_message = data.get("message", "Rate not found in response.")
-                    logger.error(
-                        f"Could not fetch rate for {symbol} from Twelve Data: {error_message}"
-                    )
-                    continue
-
-                # Convert the rate to a Decimal for precision
-                rate = Decimal(str(data["rate"]))
-                results.append((base_currency, target_currency, rate))
-
-                logger.info(f"Successfully fetched rate for {symbol} from Twelve Data.")
-
-                time.sleep(
-                    60
-                )  # We sleep every pair as to not step over TwelveData's minute limit
-
-            except requests.RequestException as e:
-                logger.error(
-                    f"Error fetching rate from Twelve Data API for symbol {symbol}: {e}"
-                )
-            except KeyError as e:
-                logger.error(
-                    f"Unexpected response structure from Twelve Data API for symbol {symbol}: Missing key {e}"
-                )
-            except Exception as e:
-                logger.error(
-                    f"An unexpected error occurred while processing Twelve Data for {symbol}: {e}"
-                )
-
-        return results
-
-
-class TwelveDataMarketsProvider(ExchangeRateProvider):
-    """
-    Provides prices for market instruments (stocks, ETFs, etc.) using the Twelve Data API.
-
-    This provider performs a multi-step process:
-    1. Parses instrument codes which can be symbols, FIGI, CUSIP, or ISIN.
-    2. For CUSIPs, it defaults the currency to USD. For all others, it searches
-       for the instrument to determine its native trading currency.
-    3. Fetches the latest price for the instrument in its native currency.
-    4. Converts the price to the requested target exchange currency.
-    """
-
-    SYMBOL_SEARCH_URL = "https://api.twelvedata.com/symbol_search"
-    PRICE_URL = "https://api.twelvedata.com/price"
-    EXCHANGE_RATE_URL = "https://api.twelvedata.com/exchange_rate"
-
-    rates_inverted = True
-
-    def __init__(self, api_key: str):
-        super().__init__(api_key)
-        self.session = requests.Session()
-
-    @classmethod
-    def requires_api_key(cls) -> bool:
-        return True
-
-    def _parse_code(self, raw_code: str) -> Tuple[str, str]:
-        """Parses the raw code to determine its type and value."""
-        if raw_code.startswith("figi:"):
-            return "figi", raw_code.removeprefix("figi:")
-        if raw_code.startswith("cusip:"):
-            return "cusip", raw_code.removeprefix("cusip:")
-        if raw_code.startswith("isin:"):
-            return "isin", raw_code.removeprefix("isin:")
-        return "symbol", raw_code
-
-    def get_rates(
-        self, target_currencies: QuerySet, exchange_currencies: set
-    ) -> List[Tuple[Currency, Currency, Decimal]]:
-        results = []
-
-        for asset in target_currencies:
-            if asset.exchange_currency not in exchange_currencies:
-                continue
-
-            code_type, code_value = self._parse_code(asset.code)
-            original_currency_code = None
-
-            try:
-                # Determine the instrument's native currency
-                if code_type == "cusip":
-                    # CUSIP codes always default to USD
-                    original_currency_code = "USD"
-                    logger.info(f"Defaulting CUSIP {code_value} to USD currency.")
-                else:
-                    # For all other types, find currency via symbol search
-                    search_params = {"symbol": code_value, "apikey": "demo"}
-                    search_res = self.session.get(
-                        self.SYMBOL_SEARCH_URL, params=search_params
-                    )
-                    search_res.raise_for_status()
-                    search_data = search_res.json()
-
-                    if not search_data.get("data"):
-                        logger.warning(
-                            f"TwelveDataMarkets: Symbol search for '{code_value}' returned no results."
-                        )
-                        continue
-
-                    instrument_data = search_data["data"][0]
-                    original_currency_code = instrument_data.get("currency")
-
-                if not original_currency_code:
-                    logger.error(
-                        f"TwelveDataMarkets: Could not determine original currency for '{code_value}'."
-                    )
-                    continue
-
-                # Get the instrument's price in its native currency
-                price_params = {code_type: code_value, "apikey": self.api_key}
-                price_res = self.session.get(self.PRICE_URL, params=price_params)
-                price_res.raise_for_status()
-                price_data = price_res.json()
-
-                if "price" not in price_data:
-                    error_message = price_data.get(
-                        "message", "Price key not found in response"
-                    )
-                    logger.error(
-                        f"TwelveDataMarkets: Could not get price for {code_type} '{code_value}': {error_message}"
-                    )
-                    continue
-
-                price_in_original_currency = Decimal(price_data["price"])
-
-                # Convert price to the target exchange currency
-                target_exchange_currency = asset.exchange_currency
-
-                if (
-                    original_currency_code.upper()
-                    == target_exchange_currency.code.upper()
-                ):
-                    final_price = price_in_original_currency
-                else:
-                    rate_symbol = (
-                        f"{original_currency_code}/{target_exchange_currency.code}"
-                    )
-                    rate_params = {"symbol": rate_symbol, "apikey": self.api_key}
-                    rate_res = self.session.get(
-                        self.EXCHANGE_RATE_URL, params=rate_params
-                    )
-                    rate_res.raise_for_status()
-                    rate_data = rate_res.json()
-
-                    if "rate" not in rate_data:
-                        error_message = rate_data.get(
-                            "message", "Rate key not found in response"
-                        )
-                        logger.error(
-                            f"TwelveDataMarkets: Could not get conversion rate for '{rate_symbol}': {error_message}"
-                        )
-                        continue
-
-                    conversion_rate = Decimal(str(rate_data["rate"]))
-                    final_price = price_in_original_currency * conversion_rate
-
-                results.append((target_exchange_currency, asset, final_price))
-                logger.info(
-                    f"Successfully processed price for {asset.code} as {final_price} {target_exchange_currency.code}"
-                )
-
-                time.sleep(
-                    60
-                )  # We sleep every pair as to not step over TwelveData's minute limit
-
-            except requests.RequestException as e:
-                logger.error(
-                    f"TwelveDataMarkets: API request failed for {code_value}: {e}"
-                )
-            except (KeyError, IndexError) as e:
-                logger.error(
-                    f"TwelveDataMarkets: Error processing API response for {code_value}: {e}"
-                )
-            except Exception as e:
-                logger.error(
-                    f"TwelveDataMarkets: An unexpected error occurred for {code_value}: {e}"
-                )
-
-        return results

app/apps/currencies/forms.py

@@ -1,15 +1,16 @@
-from apps.common.widgets.crispy.daisyui import Switch
+from crispy_bootstrap5.bootstrap5 import Switch
+from crispy_forms.bootstrap import FormActions
+from crispy_forms.helper import FormHelper
+from crispy_forms.layout import Layout, Row, Column
+from django import forms
+from django.forms import CharField
+from django.utils.translation import gettext_lazy as _
+
 from apps.common.widgets.crispy.submit import NoClassSubmit
 from apps.common.widgets.datepicker import AirDateTimePickerInput
 from apps.common.widgets.decimal import ArbitraryDecimalDisplayNumberInput
 from apps.common.widgets.tom_select import TomSelect
 from apps.currencies.models import Currency, ExchangeRate, ExchangeRateService
-from crispy_forms.bootstrap import FormActions
-from crispy_forms.helper import FormHelper
-from crispy_forms.layout import Column, Layout, Row
-from django import forms
-from django.forms import CharField
-from django.utils.translation import gettext_lazy as _
 
 
 class CurrencyForm(forms.ModelForm):
@@ -25,7 +26,6 @@ class CurrencyForm(forms.ModelForm):
             "suffix",
             "code",
             "exchange_currency",
-            "is_archived",
         ]
         widgets = {
             "exchange_currency": TomSelect(),
@@ -40,7 +40,6 @@ class CurrencyForm(forms.ModelForm):
         self.helper.layout = Layout(
             "code",
             "name",
-            Switch("is_archived"),
             "decimal_places",
             "prefix",
             "suffix",
@@ -50,13 +49,17 @@ class CurrencyForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -84,13 +87,17 @@ class ExchangeRateForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -107,7 +114,6 @@ class ExchangeRateServiceForm(forms.ModelForm):
             "fetch_interval",
             "target_currencies",
             "target_accounts",
-            "singleton",
         ]
 
     def __init__(self, *args, **kwargs):
@@ -120,11 +126,10 @@ class ExchangeRateServiceForm(forms.ModelForm):
             "name",
             "service_type",
             Switch("is_active"),
-            Switch("singleton"),
             "api_key",
             Row(
-                Column("interval_type"),
-                Column("fetch_interval"),
+                Column("interval_type", css_class="form-group col-md-6"),
+                Column("fetch_interval", css_class="form-group col-md-6"),
             ),
             "target_currencies",
             "target_accounts",
@@ -133,12 +138,16 @@ class ExchangeRateServiceForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )

app/apps/currencies/migrations/0015_exchangerate_automatic_exchangerateservice_singleton.py

@@ -1,23 +0,0 @@
-# Generated by Django 5.2.4 on 2025-08-08 02:18
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('currencies', '0014_alter_currency_options'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='exchangerate',
-            name='automatic',
-            field=models.BooleanField(default=False, verbose_name='Automatic'),
-        ),
-        migrations.AddField(
-            model_name='exchangerateservice',
-            name='singleton',
-            field=models.BooleanField(default=False, help_text='Create one exchange rate and keep updating it. Avoids database clutter.', verbose_name='Single exchange rate'),
-        ),
-    ]

app/apps/currencies/migrations/0016_alter_exchangerate_automatic.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.2.4 on 2025-08-08 02:38
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('currencies', '0015_exchangerate_automatic_exchangerateservice_singleton'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='exchangerate',
-            name='automatic',
-            field=models.BooleanField(default=False, verbose_name='Auto'),
-        ),
-    ]

app/apps/currencies/migrations/0017_alter_exchangerateservice_service_type.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.2.5 on 2025-08-16 22:18
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('currencies', '0016_alter_exchangerate_automatic'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='exchangerateservice',
-            name='service_type',
-            field=models.CharField(choices=[('synth_finance', 'Synth Finance'), ('synth_finance_stock', 'Synth Finance Stock'), ('coingecko_free', 'CoinGecko (Demo/Free)'), ('coingecko_pro', 'CoinGecko (Pro)'), ('transitive', 'Transitive (Calculated from Existing Rates)'), ('frankfurter', 'Frankfurter')], max_length=255, verbose_name='Service Type'),
-        ),
-    ]

app/apps/currencies/migrations/0018_alter_exchangerateservice_service_type.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.2.5 on 2025-08-17 03:54
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('currencies', '0017_alter_exchangerateservice_service_type'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='exchangerateservice',
-            name='service_type',
-            field=models.CharField(choices=[('synth_finance', 'Synth Finance'), ('synth_finance_stock', 'Synth Finance Stock'), ('coingecko_free', 'CoinGecko (Demo/Free)'), ('coingecko_pro', 'CoinGecko (Pro)'), ('transitive', 'Transitive (Calculated from Existing Rates)'), ('frankfurter', 'Frankfurter'), ('twelvedata', 'TwelveData')], max_length=255, verbose_name='Service Type'),
-        ),
-    ]

app/apps/currencies/migrations/0019_alter_exchangerateservice_service_type.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.2.5 on 2025-08-17 06:01
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('currencies', '0018_alter_exchangerateservice_service_type'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='exchangerateservice',
-            name='service_type',
-            field=models.CharField(choices=[('synth_finance', 'Synth Finance'), ('synth_finance_stock', 'Synth Finance Stock'), ('coingecko_free', 'CoinGecko (Demo/Free)'), ('coingecko_pro', 'CoinGecko (Pro)'), ('transitive', 'Transitive (Calculated from Existing Rates)'), ('frankfurter', 'Frankfurter'), ('twelvedata', 'TwelveData'), ('twelvedatamarkets', 'TwelveData Markets')], max_length=255, verbose_name='Service Type'),
-        ),
-    ]

app/apps/currencies/migrations/0020_migrate_synth_finance_services.py

@@ -1,51 +0,0 @@
-# Generated by Django 5.2.5 on 2025-08-17 06:25
-
-from django.db import migrations
-
-# The new value we are migrating to
-NEW_SERVICE_TYPE = "frankfurter"
-# The old values we are deprecating
-OLD_SERVICE_TYPE_TO_UPDATE = "synth_finance"
-OLD_SERVICE_TYPE_TO_DELETE = "synth_finance_stock"
-
-
-def forwards_func(apps, schema_editor):
-    """
-    Forward migration:
-    - Deletes all ExchangeRateService instances with service_type 'synth_finance_stock'.
-    - Updates all ExchangeRateService instances with service_type 'synth_finance' to 'frankfurter'.
-    """
-    ExchangeRateService = apps.get_model("currencies", "ExchangeRateService")
-    db_alias = schema_editor.connection.alias
-
-    # 1. Delete the SYNTH_FINANCE_STOCK entries
-    ExchangeRateService.objects.using(db_alias).filter(
-        service_type=OLD_SERVICE_TYPE_TO_DELETE
-    ).delete()
-
-    # 2. Update the SYNTH_FINANCE entries to FRANKFURTER
-    ExchangeRateService.objects.using(db_alias).filter(
-        service_type=OLD_SERVICE_TYPE_TO_UPDATE
-    ).update(service_type=NEW_SERVICE_TYPE, api_key=None)
-
-
-def backwards_func(apps, schema_editor):
-    """
-    Backward migration: This operation is not safely reversible.
-    - We cannot know which 'frankfurter' services were originally 'synth_finance'.
-    - The deleted 'synth_finance_stock' services cannot be recovered.
-    We will leave this function empty to allow migrating backwards without doing anything.
-    """
-    pass
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        # Add the previous migration file here
-        ("currencies", "0019_alter_exchangerateservice_service_type"),
-    ]
-
-    operations = [
-        migrations.RunPython(forwards_func, reverse_code=backwards_func),
-    ]

app/apps/currencies/migrations/0021_alter_exchangerateservice_service_type.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.2.5 on 2025-08-17 06:29
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('currencies', '0020_migrate_synth_finance_services'),
-    ]
-
-    operations = [
-        migrations.AlterField(
-            model_name='exchangerateservice',
-            name='service_type',
-            field=models.CharField(choices=[('coingecko_free', 'CoinGecko (Demo/Free)'), ('coingecko_pro', 'CoinGecko (Pro)'), ('transitive', 'Transitive (Calculated from Existing Rates)'), ('frankfurter', 'Frankfurter'), ('twelvedata', 'TwelveData'), ('twelvedatamarkets', 'TwelveData Markets')], max_length=255, verbose_name='Service Type'),
-        ),
-    ]

app/apps/currencies/migrations/0022_currency_is_archived.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.2.5 on 2025-08-30 00:47
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('currencies', '0021_alter_exchangerateservice_service_type'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='currency',
-            name='is_archived',
-            field=models.BooleanField(default=False, verbose_name='Archived'),
-        ),
-    ]

app/apps/currencies/models.py

@@ -32,11 +32,6 @@ class Currency(models.Model):
         help_text=_("Default currency for exchange calculations"),
     )
 
-    is_archived = models.BooleanField(
-        default=False,
-        verbose_name=_("Archived"),
-    )
-
     def __str__(self):
         return self.name
 
@@ -75,8 +70,6 @@ class ExchangeRate(models.Model):
     )
     date = models.DateTimeField(verbose_name=_("Date and Time"))
 
-    automatic = models.BooleanField(verbose_name=_("Auto"), default=False)
-
     class Meta:
         verbose_name = _("Exchange Rate")
         verbose_name_plural = _("Exchange Rates")
@@ -99,12 +92,11 @@ class ExchangeRateService(models.Model):
     """Configuration for exchange rate services"""
 
     class ServiceType(models.TextChoices):
+        SYNTH_FINANCE = "synth_finance", "Synth Finance"
+        SYNTH_FINANCE_STOCK = "synth_finance_stock", "Synth Finance Stock"
         COINGECKO_FREE = "coingecko_free", "CoinGecko (Demo/Free)"
         COINGECKO_PRO = "coingecko_pro", "CoinGecko (Pro)"
         TRANSITIVE = "transitive", "Transitive (Calculated from Existing Rates)"
-        FRANKFURTER = "frankfurter", "Frankfurter"
-        TWELVEDATA = "twelvedata", "TwelveData"
-        TWELVEDATA_MARKETS = "twelvedatamarkets", "TwelveData Markets"
 
     class IntervalType(models.TextChoices):
         ON = "on", _("On")
@@ -156,14 +148,6 @@ class ExchangeRateService(models.Model):
         blank=True,
     )
 
-    singleton = models.BooleanField(
-        verbose_name=_("Single exchange rate"),
-        default=False,
-        help_text=_(
-            "Create one exchange rate and keep updating it. Avoids database clutter."
-        ),
-    )
-
     class Meta:
         verbose_name = _("Exchange Rate Service")
         verbose_name_plural = _("Exchange Rate Services")

app/apps/currencies/tasks.py

@@ -8,7 +8,7 @@ logger = logging.getLogger(__name__)
 
 
 @app.periodic(cron="0 * * * *")  # Run every hour
-@app.task(lock="automatic_fetch_exchange_rates", name="automatic_fetch_exchange_rates")
+@app.task(name="automatic_fetch_exchange_rates")
 def automatic_fetch_exchange_rates(timestamp=None):
     """Fetch exchange rates for all due services"""
     fetcher = ExchangeRateFetcher()
@@ -19,7 +19,7 @@ def automatic_fetch_exchange_rates(timestamp=None):
         logger.error(e, exc_info=True)
 
 
-@app.task(lock="manual_fetch_exchange_rates", name="manual_fetch_exchange_rates")
+@app.task(name="manual_fetch_exchange_rates")
 def manual_fetch_exchange_rates(timestamp=None):
     """Fetch exchange rates for all due services"""
     fetcher = ExchangeRateFetcher()

app/apps/currencies/tests.py

@@ -40,6 +40,12 @@ class CurrencyTests(TestCase):
         with self.assertRaises(ValidationError):
             currency.full_clean()
 
+    def test_currency_unique_code(self):
+        """Test that currency codes must be unique"""
+        Currency.objects.create(code="USD", name="US Dollar", decimal_places=2)
+        with self.assertRaises(IntegrityError):
+            Currency.objects.create(code="USD", name="Another Dollar", decimal_places=2)
+
     def test_currency_unique_name(self):
         """Test that currency names must be unique"""
         Currency.objects.create(code="USD", name="US Dollar", decimal_places=2)

app/apps/dca/forms.py

@@ -1,20 +1,22 @@
+from crispy_bootstrap5.bootstrap5 import Switch, BS5Accordion
+from crispy_forms.bootstrap import FormActions, AccordionGroup
+from crispy_forms.helper import FormHelper
+from crispy_forms.layout import Layout, Row, Column, HTML
+from django import forms
+from django.utils.translation import gettext_lazy as _
+
 from apps.accounts.models import Account
+from apps.common.widgets.crispy.submit import NoClassSubmit
+from apps.common.widgets.datepicker import AirDatePickerInput
+from apps.common.widgets.decimal import ArbitraryDecimalDisplayNumberInput
+from apps.common.widgets.tom_select import TomSelect
+from apps.dca.models import DCAStrategy, DCAEntry
+from apps.common.widgets.tom_select import TransactionSelect
+from apps.transactions.models import Transaction, TransactionTag, TransactionCategory
 from apps.common.fields.forms.dynamic_select import (
     DynamicModelChoiceField,
     DynamicModelMultipleChoiceField,
 )
-from apps.common.widgets.crispy.daisyui import Switch
-from apps.common.widgets.crispy.submit import NoClassSubmit
-from apps.common.widgets.datepicker import AirDatePickerInput
-from apps.common.widgets.decimal import ArbitraryDecimalDisplayNumberInput
-from apps.common.widgets.tom_select import TomSelect, TransactionSelect
-from apps.dca.models import DCAEntry, DCAStrategy
-from apps.transactions.models import Transaction, TransactionCategory, TransactionTag
-from crispy_forms.bootstrap import AccordionGroup, FormActions, Accordion
-from crispy_forms.helper import FormHelper
-from crispy_forms.layout import HTML, Column, Layout, Row
-from django import forms
-from django.utils.translation import gettext_lazy as _
 
 
 class DCAStrategyForm(forms.ModelForm):
@@ -34,8 +36,8 @@ class DCAStrategyForm(forms.ModelForm):
         self.helper.layout = Layout(
             "name",
             Row(
-                Column("payment_currency"),
-                Column("target_currency"),
+                Column("payment_currency", css_class="form-group col-md-6"),
+                Column("target_currency", css_class="form-group col-md-6"),
             ),
             "notes",
         )
@@ -43,13 +45,17 @@ class DCAStrategyForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -149,11 +155,11 @@ class DCAEntryForm(forms.ModelForm):
         self.helper.layout = Layout(
             "date",
             Row(
-                Column("amount_paid"),
-                Column("amount_received"),
+                Column("amount_paid", css_class="form-group col-md-6"),
+                Column("amount_received", css_class="form-group col-md-6"),
             ),
             "notes",
-            Accordion(
+            BS5Accordion(
                 AccordionGroup(
                     _("Create transaction"),
                     Switch("create_transaction"),
@@ -162,11 +168,19 @@ class DCAEntryForm(forms.ModelForm):
                             Row(
                                 Column(
                                     "from_account",
+                                    css_class="form-group",
                                 ),
+                                css_class="form-row",
                             ),
                             Row(
-                                Column("from_category"),
-                                Column("from_tags"),
+                                Column(
+                                    "from_category",
+                                    css_class="form-group col-md-6 mb-0",
+                                ),
+                                Column(
+                                    "from_tags", css_class="form-group col-md-6 mb-0"
+                                ),
+                                css_class="form-row",
                             ),
                         ),
                         css_class="p-1 mx-1 my-3 border rounded-3",
@@ -178,10 +192,14 @@ class DCAEntryForm(forms.ModelForm):
                                     "to_account",
                                     css_class="form-group",
                                 ),
+                                css_class="form-row",
                             ),
                             Row(
-                                Column("to_category"),
-                                Column("to_tags"),
+                                Column(
+                                    "to_category", css_class="form-group col-md-6 mb-0"
+                                ),
+                                Column("to_tags", css_class="form-group col-md-6 mb-0"),
+                                css_class="form-row",
                             ),
                         ),
                         css_class="p-1 mx-1 my-3 border rounded-3",
@@ -202,13 +220,17 @@ class DCAEntryForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 

app/apps/export_app/forms.py

@@ -1,10 +1,11 @@
-from apps.common.widgets.crispy.submit import NoClassSubmit
 from crispy_forms.bootstrap import FormActions
 from crispy_forms.helper import FormHelper
-from crispy_forms.layout import HTML, Layout
+from crispy_forms.layout import Layout, HTML
 from django import forms
 from django.utils.translation import gettext_lazy as _
 
+from apps.common.widgets.crispy.submit import NoClassSubmit
+
 
 class ExportForm(forms.Form):
     users = forms.BooleanField(
@@ -114,7 +115,9 @@ class ExportForm(forms.Form):
             "dca",
             "import_profiles",
             FormActions(
-                NoClassSubmit("submit", _("Export"), css_class="btn btn-primary"),
+                NoClassSubmit(
+                    "submit", _("Export"), css_class="btn btn-outline-primary w-100"
+                ),
             ),
         )
 
@@ -159,7 +162,7 @@ class RestoreForm(forms.Form):
         self.helper.form_method = "post"
         self.helper.layout = Layout(
             "zip_file",
-            HTML('<hr class="hr my-3"/>'),
+            HTML("<hr />"),
             "users",
             "accounts",
             "currencies",
@@ -178,7 +181,9 @@ class RestoreForm(forms.Form):
             "dca_entries",
             "import_profiles",
             FormActions(
-                NoClassSubmit("submit", _("Restore"), css_class="btn btn-primary"),
+                NoClassSubmit(
+                    "submit", _("Restore"), css_class="btn btn-outline-primary w-100"
+                ),
             ),
         )
 

app/apps/import_app/forms.py

@@ -1,5 +1,3 @@
-from apps.common.widgets.crispy.submit import NoClassSubmit
-from apps.import_app.models import ImportProfile
 from crispy_forms.bootstrap import FormActions
 from crispy_forms.helper import FormHelper
 from crispy_forms.layout import (
@@ -8,6 +6,9 @@ from crispy_forms.layout import (
 from django import forms
 from django.utils.translation import gettext_lazy as _
 
+from apps.import_app.models import ImportProfile
+from apps.common.widgets.crispy.submit import NoClassSubmit
+
 
 class ImportProfileForm(forms.ModelForm):
     class Meta:
@@ -29,13 +30,17 @@ class ImportProfileForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -52,6 +57,8 @@ class ImportRunFileUploadForm(forms.Form):
         self.helper.layout = Layout(
             "file",
             FormActions(
-                NoClassSubmit("submit", _("Import"), css_class="btn btn-primary"),
+                NoClassSubmit(
+                    "submit", _("Import"), css_class="btn btn-outline-primary w-100"
+                ),
             ),
         )

app/apps/import_app/services/v1.py

@@ -459,13 +459,12 @@ class ImportService:
                 # Build query conditions for each field in the rule
                 for field in rule.fields:
                     if field in transaction_data:
-                        value = transaction_data[field]
-                        # Use __iexact only for string fields; non-string types
-                        # (date, Decimal, bool, int, etc.) don't support UPPER()
-                        if rule.match_type == "strict" or not isinstance(value, str):
-                            query = query.filter(**{field: value})
-                        else:  # lax matching for strings only
-                            query = query.filter(**{f"{field}__iexact": value})
+                        if rule.match_type == "strict":
+                            query = query.filter(**{field: transaction_data[field]})
+                        else:  # lax matching
+                            query = query.filter(
+                                **{f"{field}__iexact": transaction_data[field]}
+                            )
 
                 # If we found any matching transaction, it's a duplicate
                 if query.exists():
@@ -476,27 +475,11 @@ class ImportService:
     def _coerce_type(
         self, value: str, mapping: version_1.ColumnMapping
     ) -> Union[str, int, bool, Decimal, datetime, list, None]:
-        coerce_to = mapping.coerce_to
-
-        # Handle detection methods that don't require a source value
-        if coerce_to == "transaction_type" and isinstance(
-            mapping, version_1.TransactionTypeMapping
-        ):
-            if mapping.detection_method == "always_income":
-                return Transaction.Type.INCOME
-            elif mapping.detection_method == "always_expense":
-                return Transaction.Type.EXPENSE
-        elif coerce_to == "is_paid" and isinstance(
-            mapping, version_1.TransactionIsPaidMapping
-        ):
-            if mapping.detection_method == "always_paid":
-                return True
-            elif mapping.detection_method == "always_unpaid":
-                return False
-
         if not value:
             return None
 
+        coerce_to = mapping.coerce_to
+
         return self._coerce_single_type(value, coerce_to, mapping)
 
     @staticmethod

app/apps/import_app/tests.py

@@ -0,0 +1,3 @@
+from django.test import TestCase
+
+# Create your tests here.

app/apps/import_app/tests/test_import_service_v1.py

@@ -1,276 +0,0 @@
-"""
-Tests for ImportService v1, specifically for deduplication logic.
-
-These tests verify that the _check_duplicate_transaction method handles
-different field types correctly, particularly ensuring that __iexact
-is only used for string fields (not dates, decimals, etc.).
-"""
-
-from datetime import date
-from decimal import Decimal
-from unittest.mock import MagicMock, patch
-
-from django.test import TestCase
-
-from apps.accounts.models import Account, AccountGroup
-from apps.currencies.models import Currency
-from apps.import_app.models import ImportProfile, ImportRun
-from apps.import_app.services.v1 import ImportService
-from apps.transactions.models import Transaction
-
-
-class DeduplicationTests(TestCase):
-    """Tests for transaction deduplication during import."""
-
-    def setUp(self):
-        """Set up test data."""
-        self.currency = Currency.objects.create(
-            code="USD", name="US Dollar", decimal_places=2, prefix="$ "
-        )
-        self.account_group = AccountGroup.objects.create(name="Test Group")
-        self.account = Account.objects.create(
-            name="Test Account", group=self.account_group, currency=self.currency
-        )
-
-        # Create an existing transaction for deduplication tests
-        self.existing_transaction = Transaction.objects.create(
-            account=self.account,
-            type=Transaction.Type.EXPENSE,
-            date=date(2024, 1, 15),
-            amount=Decimal("100.00"),
-            description="Existing Transaction",
-            internal_id="ABC123",
-        )
-
-    def _create_import_service_with_deduplication(
-        self, fields: list[str], match_type: str = "lax"
-    ) -> ImportService:
-        """Helper to create an ImportService with specific deduplication rules."""
-        yaml_config = f"""
-settings:
-  file_type: csv
-  importing: transactions
-  trigger_transaction_rules: false
-mapping:
-  date_field:
-    source: date
-    target: date
-    format: "%Y-%m-%d"
-  amount_field:
-    source: amount
-    target: amount
-  description_field:
-    source: description
-    target: description
-  account_field:
-    source: account
-    target: account
-    type: id
-deduplication:
-  - type: compare
-    fields: {fields}
-    match_type: {match_type}
-"""
-        profile = ImportProfile.objects.create(
-            name=f"Test Profile {match_type} {'_'.join(fields)}",
-            yaml_config=yaml_config,
-            version=ImportProfile.Versions.VERSION_1,
-        )
-        import_run = ImportRun.objects.create(
-            profile=profile,
-            file_name="test.csv",
-        )
-        return ImportService(import_run)
-
-    def test_deduplication_with_date_field_strict_match(self):
-        """Test that date fields work with strict matching."""
-        service = self._create_import_service_with_deduplication(
-            fields=["date"], match_type="strict"
-        )
-
-        # Should find duplicate when date matches
-        is_duplicate = service._check_duplicate_transaction({"date": date(2024, 1, 15)})
-        self.assertTrue(is_duplicate)
-
-        # Should not find duplicate when date differs
-        is_duplicate = service._check_duplicate_transaction({"date": date(2024, 2, 20)})
-        self.assertFalse(is_duplicate)
-
-    def test_deduplication_with_date_field_lax_match(self):
-        """
-        Test that date fields use strict matching even when match_type is 'lax'.
-
-        This is the fix for the UPPER(date) PostgreSQL error. Date fields
-        cannot use __iexact, so they should fall back to strict matching.
-        """
-        service = self._create_import_service_with_deduplication(
-            fields=["date"], match_type="lax"
-        )
-
-        # Should find duplicate when date matches (using strict comparison)
-        is_duplicate = service._check_duplicate_transaction({"date": date(2024, 1, 15)})
-        self.assertTrue(is_duplicate)
-
-        # Should not find duplicate when date differs
-        is_duplicate = service._check_duplicate_transaction({"date": date(2024, 2, 20)})
-        self.assertFalse(is_duplicate)
-
-    def test_deduplication_with_amount_field_lax_match(self):
-        """
-        Test that Decimal fields use strict matching even when match_type is 'lax'.
-
-        Decimal fields cannot use __iexact, so they should fall back to strict matching.
-        """
-        service = self._create_import_service_with_deduplication(
-            fields=["amount"], match_type="lax"
-        )
-
-        # Should find duplicate when amount matches
-        is_duplicate = service._check_duplicate_transaction(
-            {"amount": Decimal("100.00")}
-        )
-        self.assertTrue(is_duplicate)
-
-        # Should not find duplicate when amount differs
-        is_duplicate = service._check_duplicate_transaction(
-            {"amount": Decimal("200.00")}
-        )
-        self.assertFalse(is_duplicate)
-
-    def test_deduplication_with_string_field_lax_match(self):
-        """
-        Test that string fields use case-insensitive matching with match_type 'lax'.
-        """
-        service = self._create_import_service_with_deduplication(
-            fields=["description"], match_type="lax"
-        )
-
-        # Should find duplicate with case-insensitive match
-        is_duplicate = service._check_duplicate_transaction(
-            {"description": "EXISTING TRANSACTION"}
-        )
-        self.assertTrue(is_duplicate)
-
-        # Should find duplicate with exact case match
-        is_duplicate = service._check_duplicate_transaction(
-            {"description": "Existing Transaction"}
-        )
-        self.assertTrue(is_duplicate)
-
-        # Should not find duplicate when description differs
-        is_duplicate = service._check_duplicate_transaction(
-            {"description": "Different Transaction"}
-        )
-        self.assertFalse(is_duplicate)
-
-    def test_deduplication_with_string_field_strict_match(self):
-        """
-        Test that string fields use case-sensitive matching with match_type 'strict'.
-        """
-        service = self._create_import_service_with_deduplication(
-            fields=["description"], match_type="strict"
-        )
-
-        # Should NOT find duplicate with different case (strict matching)
-        is_duplicate = service._check_duplicate_transaction(
-            {"description": "EXISTING TRANSACTION"}
-        )
-        self.assertFalse(is_duplicate)
-
-        # Should find duplicate with exact case match
-        is_duplicate = service._check_duplicate_transaction(
-            {"description": "Existing Transaction"}
-        )
-        self.assertTrue(is_duplicate)
-
-    def test_deduplication_with_multiple_fields_mixed_types(self):
-        """
-        Test deduplication with multiple fields of different types.
-
-        Verifies that string fields use __iexact while non-string fields
-        use strict matching, all in the same deduplication rule.
-        """
-        service = self._create_import_service_with_deduplication(
-            fields=["date", "amount", "description"], match_type="lax"
-        )
-
-        # Should find duplicate when all fields match (with case-insensitive description)
-        is_duplicate = service._check_duplicate_transaction(
-            {
-                "date": date(2024, 1, 15),
-                "amount": Decimal("100.00"),
-                "description": "existing transaction",  # lowercase should match
-            }
-        )
-        self.assertTrue(is_duplicate)
-
-        # Should NOT find duplicate when date differs
-        is_duplicate = service._check_duplicate_transaction(
-            {
-                "date": date(2024, 2, 20),
-                "amount": Decimal("100.00"),
-                "description": "existing transaction",
-            }
-        )
-        self.assertFalse(is_duplicate)
-
-        # Should NOT find duplicate when amount differs
-        is_duplicate = service._check_duplicate_transaction(
-            {
-                "date": date(2024, 1, 15),
-                "amount": Decimal("999.99"),
-                "description": "existing transaction",
-            }
-        )
-        self.assertFalse(is_duplicate)
-
-    def test_deduplication_with_internal_id_lax_match(self):
-        """Test deduplication with internal_id field using lax matching."""
-        service = self._create_import_service_with_deduplication(
-            fields=["internal_id"], match_type="lax"
-        )
-
-        # Should find duplicate with case-insensitive match
-        is_duplicate = service._check_duplicate_transaction(
-            {"internal_id": "abc123"}  # lowercase should match ABC123
-        )
-        self.assertTrue(is_duplicate)
-
-        # Should find duplicate with exact match
-        is_duplicate = service._check_duplicate_transaction({"internal_id": "ABC123"})
-        self.assertTrue(is_duplicate)
-
-        # Should not find duplicate when internal_id differs
-        is_duplicate = service._check_duplicate_transaction({"internal_id": "XYZ789"})
-        self.assertFalse(is_duplicate)
-
-    def test_no_duplicate_when_no_transactions_exist(self):
-        """Test that no duplicate is found when there are no matching transactions."""
-        # Hard delete to bypass signals that require user context
-        self.existing_transaction.hard_delete()
-
-        service = self._create_import_service_with_deduplication(
-            fields=["date", "amount"], match_type="lax"
-        )
-
-        is_duplicate = service._check_duplicate_transaction(
-            {
-                "date": date(2024, 1, 15),
-                "amount": Decimal("100.00"),
-            }
-        )
-        self.assertFalse(is_duplicate)
-
-    def test_deduplication_with_missing_field_in_data(self):
-        """Test that missing fields in transaction_data are handled gracefully."""
-        service = self._create_import_service_with_deduplication(
-            fields=["date", "nonexistent_field"], match_type="lax"
-        )
-
-        # Should still work, only checking the fields that exist
-        is_duplicate = service._check_duplicate_transaction(
-            {
-                "date": date(2024, 1, 15),
-            }
-        )
-        self.assertTrue(is_duplicate)

app/apps/insights/forms.py

@@ -1,15 +1,16 @@
-from apps.common.widgets.datepicker import (
-    AirDatePickerInput,
-    AirMonthYearPickerInput,
-    AirYearPickerInput,
-)
-from apps.common.widgets.tom_select import TomSelect
-from apps.transactions.models import TransactionCategory
 from crispy_forms.helper import FormHelper
-from crispy_forms.layout import Column, Field, Layout, Row
+from crispy_forms.layout import Layout, Field, Row, Column
 from django import forms
 from django.utils.translation import gettext_lazy as _
 
+from apps.common.widgets.datepicker import (
+    AirMonthYearPickerInput,
+    AirYearPickerInput,
+    AirDatePickerInput,
+)
+from apps.transactions.models import TransactionCategory
+from apps.common.widgets.tom_select import TomSelect
+
 
 class SingleMonthForm(forms.Form):
     month = forms.DateField(
@@ -58,8 +59,8 @@ class MonthRangeForm(forms.Form):
 
         self.helper.layout = Layout(
             Row(
-                Column("month_from"),
-                Column("month_to"),
+                Column("month_from", css_class="form-group col-md-6"),
+                Column("month_to", css_class="form-group col-md-6"),
             ),
         )
 
@@ -81,8 +82,8 @@ class YearRangeForm(forms.Form):
 
         self.helper.layout = Layout(
             Row(
-                Column("year_from"),
-                Column("year_to"),
+                Column("year_from", css_class="form-group col-md-6"),
+                Column("year_to", css_class="form-group col-md-6"),
             ),
         )
 
@@ -104,8 +105,8 @@ class DateRangeForm(forms.Form):
 
         self.helper.layout = Layout(
             Row(
-                Column("date_from"),
-                Column("date_to"),
+                Column("date_from", css_class="form-group col-md-6"),
+                Column("date_to", css_class="form-group col-md-6"),
                 css_class="mb-0",
             ),
         )

app/apps/insights/utils/category_overview.py

@@ -9,13 +9,8 @@ from apps.currencies.models import Currency
 from apps.currencies.utils.convert import convert
 
 
-def get_categories_totals(
-    transactions_queryset, ignore_empty=False, show_entities=False
-):
-    # Step 1: Aggregate transaction data by category and currency.
-    # This query calculates the total current and projected income/expense for each
-    # category by grouping transactions and summing up their amounts based on their
-    # type (income/expense) and payment status (paid/unpaid).
+def get_categories_totals(transactions_queryset, ignore_empty=False):
+    # First get the category totals as before
     category_currency_metrics = (
         transactions_queryset.values(
             "category",
@@ -79,10 +74,7 @@ def get_categories_totals(
         .order_by("category__name")
     )
 
-    # Step 2: Aggregate transaction data by tag, category, and currency.
-    # This is similar to the category metrics but adds tags to the grouping,
-    # allowing for a breakdown of totals by tag within each category. It also
-    # handles untagged transactions, where the 'tags' field is None.
+    # Get tag totals within each category with currency details
     tag_metrics = transactions_queryset.values(
         "category",
         "tags",
@@ -137,12 +129,10 @@ def get_categories_totals(
         ),
     )
 
-    # Step 3: Initialize the main dictionary to structure the final results.
-    # The data will be organized hierarchically: category -> currency -> tags -> entities.
+    # Process the results to structure by category
     result = {}
 
-    # Step 4: Process the aggregated category metrics to build the initial result structure.
-    # This loop iterates through each category's metrics and populates the `result` dict.
+    # Process category totals first
     for metric in category_currency_metrics:
         # Skip empty categories if ignore_empty is True
         if ignore_empty and all(
@@ -193,7 +183,7 @@ def get_categories_totals(
             "total_final": total_final,
         }
 
-        # Step 4a: Handle currency conversion for category totals if an exchange currency is defined.
+        # Add exchanged values if exchange_currency exists
         if metric["account__currency__exchange_currency"]:
             from_currency = Currency.objects.get(id=currency_id)
             exchange_currency = Currency.objects.get(
@@ -232,7 +222,7 @@ def get_categories_totals(
 
         result[category_id]["currencies"][currency_id] = currency_data
 
-    # Step 5: Process the aggregated tag metrics and integrate them into the result structure.
+    # Process tag totals and add them to the result, including untagged
     for tag_metric in tag_metrics:
         category_id = tag_metric["category"]
         tag_id = tag_metric["tags"]  # Will be None for untagged transactions
@@ -250,7 +240,6 @@ def get_categories_totals(
                 result[category_id]["tags"][tag_key] = {
                     "name": tag_name,
                     "currencies": {},
-                    "entities": {},
                 }
 
             currency_id = tag_metric["account__currency"]
@@ -289,7 +278,7 @@ def get_categories_totals(
                 "total_final": tag_total_final,
             }
 
-            # Step 5a: Handle currency conversion for tag totals.
+            # Add exchange currency support for tags
             if tag_metric["account__currency__exchange_currency"]:
                 from_currency = Currency.objects.get(id=currency_id)
                 exchange_currency = Currency.objects.get(
@@ -330,175 +319,4 @@ def get_categories_totals(
                 currency_id
             ] = tag_currency_data
 
-    # Step 6: If requested, aggregate and process entity-level data.
-    if show_entities:
-        entity_metrics = transactions_queryset.values(
-            "category",
-            "tags",
-            "entities",
-            "entities__name",
-            "account__currency",
-            "account__currency__code",
-            "account__currency__name",
-            "account__currency__decimal_places",
-            "account__currency__prefix",
-            "account__currency__suffix",
-            "account__currency__exchange_currency",
-        ).annotate(
-            expense_current=Coalesce(
-                Sum(
-                    Case(
-                        When(
-                            type=Transaction.Type.EXPENSE, is_paid=True, then="amount"
-                        ),
-                        default=Value(0),
-                        output_field=models.DecimalField(),
-                    )
-                ),
-                Decimal("0"),
-            ),
-            expense_projected=Coalesce(
-                Sum(
-                    Case(
-                        When(
-                            type=Transaction.Type.EXPENSE, is_paid=False, then="amount"
-                        ),
-                        default=Value(0),
-                        output_field=models.DecimalField(),
-                    )
-                ),
-                Decimal("0"),
-            ),
-            income_current=Coalesce(
-                Sum(
-                    Case(
-                        When(type=Transaction.Type.INCOME, is_paid=True, then="amount"),
-                        default=Value(0),
-                        output_field=models.DecimalField(),
-                    )
-                ),
-                Decimal("0"),
-            ),
-            income_projected=Coalesce(
-                Sum(
-                    Case(
-                        When(
-                            type=Transaction.Type.INCOME, is_paid=False, then="amount"
-                        ),
-                        default=Value(0),
-                        output_field=models.DecimalField(),
-                    )
-                ),
-                Decimal("0"),
-            ),
-        )
-
-        for entity_metric in entity_metrics:
-            category_id = entity_metric["category"]
-            tag_id = entity_metric["tags"]
-            entity_id = entity_metric["entities"]
-
-            if category_id in result:
-                tag_key = tag_id if tag_id is not None else "untagged"
-                if tag_key in result[category_id]["tags"]:
-                    entity_key = entity_id if entity_id is not None else "no_entity"
-                    entity_name = (
-                        entity_metric["entities__name"]
-                        if entity_id is not None
-                        else None
-                    )
-
-                    if "entities" not in result[category_id]["tags"][tag_key]:
-                        result[category_id]["tags"][tag_key]["entities"] = {}
-
-                    if (
-                        entity_key
-                        not in result[category_id]["tags"][tag_key]["entities"]
-                    ):
-                        result[category_id]["tags"][tag_key]["entities"][entity_key] = {
-                            "name": entity_name,
-                            "currencies": {},
-                        }
-
-                    currency_id = entity_metric["account__currency"]
-
-                    entity_total_current = (
-                        entity_metric["income_current"]
-                        - entity_metric["expense_current"]
-                    )
-                    entity_total_projected = (
-                        entity_metric["income_projected"]
-                        - entity_metric["expense_projected"]
-                    )
-                    entity_total_income = (
-                        entity_metric["income_current"]
-                        + entity_metric["income_projected"]
-                    )
-                    entity_total_expense = (
-                        entity_metric["expense_current"]
-                        + entity_metric["expense_projected"]
-                    )
-                    entity_total_final = entity_total_current + entity_total_projected
-
-                    entity_currency_data = {
-                        "currency": {
-                            "code": entity_metric["account__currency__code"],
-                            "name": entity_metric["account__currency__name"],
-                            "decimal_places": entity_metric[
-                                "account__currency__decimal_places"
-                            ],
-                            "prefix": entity_metric["account__currency__prefix"],
-                            "suffix": entity_metric["account__currency__suffix"],
-                        },
-                        "expense_current": entity_metric["expense_current"],
-                        "expense_projected": entity_metric["expense_projected"],
-                        "total_expense": entity_total_expense,
-                        "income_current": entity_metric["income_current"],
-                        "income_projected": entity_metric["income_projected"],
-                        "total_income": entity_total_income,
-                        "total_current": entity_total_current,
-                        "total_projected": entity_total_projected,
-                        "total_final": entity_total_final,
-                    }
-
-                    if entity_metric["account__currency__exchange_currency"]:
-                        from_currency = Currency.objects.get(id=currency_id)
-                        exchange_currency = Currency.objects.get(
-                            id=entity_metric["account__currency__exchange_currency"]
-                        )
-
-                        exchanged = {}
-                        for field in [
-                            "expense_current",
-                            "expense_projected",
-                            "income_current",
-                            "income_projected",
-                            "total_income",
-                            "total_expense",
-                            "total_current",
-                            "total_projected",
-                            "total_final",
-                        ]:
-                            amount, prefix, suffix, decimal_places = convert(
-                                amount=entity_currency_data[field],
-                                from_currency=from_currency,
-                                to_currency=exchange_currency,
-                            )
-                            if amount is not None:
-                                exchanged[field] = amount
-                                if "currency" not in exchanged:
-                                    exchanged["currency"] = {
-                                        "prefix": prefix,
-                                        "suffix": suffix,
-                                        "decimal_places": decimal_places,
-                                        "code": exchange_currency.code,
-                                        "name": exchange_currency.name,
-                                    }
-                        if exchanged:
-                            entity_currency_data["exchanged"] = exchanged
-
-                    result[category_id]["tags"][tag_key]["entities"][entity_key][
-                        "currencies"
-                    ][currency_id] = entity_currency_data
-
     return result

app/apps/insights/utils/transactions.py

@@ -13,9 +13,7 @@ from apps.insights.forms import (
 )
 
 
-def get_transactions(
-    request, include_unpaid=True, include_silent=False, include_untracked_accounts=False
-):
+def get_transactions(request, include_unpaid=True, include_silent=False):
     transactions = Transaction.objects.all()
 
     filter_type = request.GET.get("type", None)
@@ -97,11 +95,4 @@ def get_transactions(
             Q(Q(category__mute=True) & ~Q(category=None)) | Q(mute=True)
         )
 
-    if not include_untracked_accounts:
-        transactions = transactions.exclude(
-            account__in=request.user.untracked_accounts.all()
-        )
-
-    transactions = transactions.exclude(account__currency__is_archived=True)
-
     return transactions

app/apps/insights/views.py

@@ -74,9 +74,7 @@ def index(request):
 def sankey_by_account(request):
     # Get filtered transactions
 
-    transactions = get_transactions(
-        request, include_untracked_accounts=True, include_silent=True
-    )
+    transactions = get_transactions(request)
 
     # Generate Sankey data
     sankey_data = generate_sankey_data_by_account(transactions)
@@ -93,9 +91,7 @@ def sankey_by_account(request):
 @require_http_methods(["GET"])
 def sankey_by_currency(request):
     # Get filtered transactions
-    transactions = get_transactions(
-        request, include_silent=True, include_untracked_accounts=True
-    )
+    transactions = get_transactions(request)
 
     # Generate Sankey data
     sankey_data = generate_sankey_data_by_currency(transactions)
@@ -184,14 +180,6 @@ def category_overview(request):
     else:
         show_tags = request.session.get("insights_category_explorer_show_tags", True)
 
-    if "show_entities" in request.GET:
-        show_entities = request.GET["show_entities"] == "on"
-        request.session["insights_category_explorer_show_entities"] = show_entities
-    else:
-        show_entities = request.session.get(
-            "insights_category_explorer_show_entities", False
-        )
-
     if "showing" in request.GET:
         showing = request.GET["showing"]
         request.session["insights_category_explorer_showing"] = showing
@@ -202,9 +190,7 @@ def category_overview(request):
     transactions = get_transactions(request, include_silent=True)
 
     total_table = get_categories_totals(
-        transactions_queryset=transactions,
-        ignore_empty=False,
-        show_entities=show_entities,
+        transactions_queryset=transactions, ignore_empty=False
     )
 
     return render(
@@ -214,7 +200,6 @@ def category_overview(request):
             "total_table": total_table,
             "view_type": view_type,
             "show_tags": show_tags,
-            "show_entities": show_entities,
             "showing": showing,
         },
     )
@@ -254,14 +239,10 @@ def late_transactions(request):
 @login_required
 @require_http_methods(["GET"])
 def emergency_fund(request):
-    transactions_currency_queryset = (
-        Transaction.objects.filter(
-            is_paid=True, account__is_archived=False, account__is_asset=False
-        )
-        .exclude(account__in=request.user.untracked_accounts.all())
-        .order_by(
-            "account__currency__name",
-        )
+    transactions_currency_queryset = Transaction.objects.filter(
+        is_paid=True, account__is_archived=False, account__is_asset=False
+    ).order_by(
+        "account__currency__name",
     )
     currency_net_worth = calculate_currency_totals(
         transactions_queryset=transactions_currency_queryset, ignore_empty=False
@@ -281,7 +262,6 @@ def emergency_fund(request):
             category__mute=False,
             mute=False,
         )
-        .exclude(account__in=request.user.untracked_accounts.all())
         .values("reference_date", "account__currency")
         .annotate(monthly_total=Sum("amount"))
     )

app/apps/monthly_overview/views.py

@@ -107,15 +107,9 @@ def transactions_list(request, month: int, year: int):
 @require_http_methods(["GET"])
 def monthly_summary(request, month: int, year: int):
     # Base queryset with all required filters
-    base_queryset = (
-        Transaction.objects.filter(
-            reference_date__year=year,
-            reference_date__month=month,
-            account__is_asset=False,
-        )
-        .exclude(Q(Q(category__mute=True) & ~Q(category=None)) | Q(mute=True))
-        .exclude(account__in=request.user.untracked_accounts.all())
-    )
+    base_queryset = Transaction.objects.filter(
+        reference_date__year=year, reference_date__month=month, account__is_asset=False
+    ).exclude(Q(Q(category__mute=True) & ~Q(category=None)) | Q(mute=True))
 
     data = calculate_currency_totals(base_queryset, ignore_empty=True)
     percentages = calculate_percentage_distribution(data)
@@ -171,14 +165,10 @@ def monthly_account_summary(request, month: int, year: int):
 @require_http_methods(["GET"])
 def monthly_currency_summary(request, month: int, year: int):
     # Base queryset with all required filters
-    base_queryset = (
-        Transaction.objects.filter(
-            reference_date__year=year,
-            reference_date__month=month,
-        )
-        .exclude(Q(Q(category__mute=True) & ~Q(category=None)) | Q(mute=True))
-        .exclude(account__in=request.user.untracked_accounts.all())
-    )
+    base_queryset = Transaction.objects.filter(
+        reference_date__year=year,
+        reference_date__month=month,
+    ).exclude(Q(Q(category__mute=True) & ~Q(category=None)) | Q(mute=True))
 
     currency_data = calculate_currency_totals(base_queryset.all(), ignore_empty=True)
     currency_percentages = calculate_percentage_distribution(currency_data)

app/apps/net_worth/utils/calculate_net_worth.py

@@ -30,7 +30,6 @@ def calculate_historical_currency_net_worth(queryset):
             | Q(accounts__visibility="private", accounts__owner=None),
             accounts__is_archived=False,
             accounts__isnull=False,
-            is_archived=False,
         )
         .values_list("name", flat=True)
         .distinct()
@@ -182,29 +181,3 @@ def calculate_historical_account_balance(queryset):
         historical_account_balance[date_filter(end_date, "b Y")] = month_data
 
     return historical_account_balance
-
-
-def calculate_monthly_net_worth_difference(historical_net_worth):
-    diff_dict = OrderedDict()
-    if not historical_net_worth:
-        return diff_dict
-
-    # Get all currencies
-    currencies = set()
-    for data in historical_net_worth.values():
-        currencies.update(data.keys())
-
-    # Initialize prev_values for all currencies
-    prev_values = {currency: Decimal("0.00") for currency in currencies}
-
-    for month, values in historical_net_worth.items():
-        diff_values = {}
-        for currency in sorted(list(currencies)):
-            current_val = values.get(currency, Decimal("0.00"))
-            prev_val = prev_values.get(currency, Decimal("0.00"))
-            diff_values[currency] = current_val - prev_val
-
-        diff_dict[month] = diff_values
-        prev_values = values.copy()
-
-    return diff_dict

app/apps/net_worth/views.py

@@ -8,7 +8,6 @@ from django.views.decorators.http import require_http_methods
 from apps.net_worth.utils.calculate_net_worth import (
     calculate_historical_currency_net_worth,
     calculate_historical_account_balance,
-    calculate_monthly_net_worth_difference,
 )
 from apps.transactions.models import Transaction
 from apps.transactions.utils.calculations import (
@@ -21,18 +20,17 @@ from apps.transactions.utils.calculations import (
 @require_http_methods(["GET"])
 def net_worth(request):
     if "view_type" in request.GET:
+        print(request.GET["view_type"])
         view_type = request.GET["view_type"]
         request.session["networth_view_type"] = view_type
     else:
         view_type = request.session.get("networth_view_type", "current")
 
     if view_type == "current":
-        transactions_currency_queryset = (
-            Transaction.objects.filter(is_paid=True, account__is_archived=False)
-            .order_by(
-                "account__currency__name",
-            )
-            .exclude(account__in=request.user.untracked_accounts.all())
+        transactions_currency_queryset = Transaction.objects.filter(
+            is_paid=True, account__is_archived=False
+        ).order_by(
+            "account__currency__name",
         )
         transactions_account_queryset = Transaction.objects.filter(
             is_paid=True, account__is_archived=False
@@ -41,12 +39,10 @@ def net_worth(request):
             "account__name",
         )
     else:
-        transactions_currency_queryset = (
-            Transaction.objects.filter(account__is_archived=False)
-            .order_by(
-                "account__currency__name",
-            )
-            .exclude(account__in=request.user.untracked_accounts.all())
+        transactions_currency_queryset = Transaction.objects.filter(
+            account__is_archived=False
+        ).order_by(
+            "account__currency__name",
         )
         transactions_account_queryset = Transaction.objects.filter(
             account__is_archived=False
@@ -97,38 +93,6 @@ def net_worth(request):
 
     chart_data_currency_json = json.dumps(chart_data_currency, cls=DjangoJSONEncoder)
 
-    monthly_difference_data = calculate_monthly_net_worth_difference(
-        historical_net_worth=historical_currency_net_worth
-    )
-
-    diff_labels = (
-        list(monthly_difference_data.keys()) if monthly_difference_data else []
-    )
-    diff_currencies = (
-        list(monthly_difference_data[diff_labels[0]].keys())
-        if monthly_difference_data and diff_labels
-        else []
-    )
-
-    diff_datasets = []
-    for i, currency in enumerate(diff_currencies):
-        data = [
-            float(month_data.get(currency, 0))
-            for month_data in monthly_difference_data.values()
-        ]
-        diff_datasets.append(
-            {
-                "label": currency,
-                "data": data,
-                "borderWidth": 3,
-            }
-        )
-
-    chart_data_monthly_difference = {"labels": diff_labels, "datasets": diff_datasets}
-    chart_data_monthly_difference_json = json.dumps(
-        chart_data_monthly_difference, cls=DjangoJSONEncoder
-    )
-
     historical_account_balance = calculate_historical_account_balance(
         queryset=transactions_account_queryset
     )
@@ -173,7 +137,6 @@ def net_worth(request):
             "chart_data_accounts_json": chart_data_accounts_json,
             "accounts": accounts,
             "type": view_type,
-            "chart_data_monthly_difference_json": chart_data_monthly_difference_json,
         },
     )
 

app/apps/rules/forms.py

@@ -1,22 +1,16 @@
-from crispy_forms.bootstrap import Alert
-from apps.common.fields.forms.dynamic_select import DynamicModelChoiceField
-from apps.common.widgets.crispy.daisyui import Switch
-from apps.common.widgets.crispy.submit import NoClassSubmit
-from apps.common.widgets.tom_select import TomSelect, TransactionSelect
-from apps.rules.models import (
-    TransactionRule,
-    TransactionRuleAction,
-    UpdateOrCreateTransactionRuleAction,
-)
-from apps.transactions.forms import BulkEditTransactionForm
-from apps.transactions.models import Transaction
-from crispy_forms.bootstrap import AccordionGroup, FormActions, Accordion
+from crispy_bootstrap5.bootstrap5 import Switch, BS5Accordion
+from crispy_forms.bootstrap import FormActions, AccordionGroup
 from crispy_forms.helper import FormHelper
-from crispy_forms.layout import HTML, Column, Field, Layout, Row
+from crispy_forms.layout import Layout, Field, Row, Column
 from django import forms
 from django.core.exceptions import ValidationError
 from django.utils.translation import gettext_lazy as _
 
+from apps.common.widgets.crispy.submit import NoClassSubmit
+from apps.common.widgets.tom_select import TomSelect
+from apps.rules.models import TransactionRule, UpdateOrCreateTransactionRuleAction
+from apps.rules.models import TransactionRuleAction
+
 
 class TransactionRuleForm(forms.ModelForm):
     class Meta:
@@ -37,6 +31,7 @@ class TransactionRuleForm(forms.ModelForm):
         self.helper = FormHelper()
         self.helper.form_tag = False
         self.helper.form_method = "post"
+        # TO-DO: Add helper with available commands
         self.helper.layout = Layout(
             Switch("active"),
             "name",
@@ -45,25 +40,24 @@ class TransactionRuleForm(forms.ModelForm):
                 Column(Switch("on_create")),
                 Column(Switch("on_delete")),
             ),
-            "order",
-            Switch("sequenced"),
             "description",
             "trigger",
-            Alert(
-                _("You can add actions to this rule in the next screen."), dismiss=False
-            ),
         )
 
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -71,11 +65,10 @@ class TransactionRuleForm(forms.ModelForm):
 class TransactionRuleActionForm(forms.ModelForm):
     class Meta:
         model = TransactionRuleAction
-        fields = ("value", "field", "order")
+        fields = ("value", "field")
         labels = {
             "field": _("Set field"),
             "value": _("To"),
-            "order": _("Order"),
         }
         widgets = {"field": TomSelect(clear_button=False)}
 
@@ -89,7 +82,6 @@ class TransactionRuleActionForm(forms.ModelForm):
         self.helper.form_method = "post"
         # TO-DO: Add helper with available commands
         self.helper.layout = Layout(
-            "order",
             "field",
             "value",
         )
@@ -97,13 +89,17 @@ class TransactionRuleActionForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -151,11 +147,9 @@ class UpdateOrCreateTransactionRuleActionForm(forms.ModelForm):
             "search_category_operator": TomSelect(clear_button=False),
             "search_internal_note_operator": TomSelect(clear_button=False),
             "search_internal_id_operator": TomSelect(clear_button=False),
-            "search_mute_operator": TomSelect(clear_button=False),
         }
 
         labels = {
-            "order": _("Order"),
             "search_account_operator": _("Operator"),
             "search_type_operator": _("Operator"),
             "search_is_paid_operator": _("Operator"),
@@ -169,7 +163,6 @@ class UpdateOrCreateTransactionRuleActionForm(forms.ModelForm):
             "search_internal_id_operator": _("Operator"),
             "search_tags_operator": _("Operator"),
             "search_entities_operator": _("Operator"),
-            "search_mute_operator": _("Operator"),
             "search_account": _("Account"),
             "search_type": _("Type"),
             "search_is_paid": _("Paid"),
@@ -183,7 +176,6 @@ class UpdateOrCreateTransactionRuleActionForm(forms.ModelForm):
             "search_internal_id": _("Internal ID"),
             "search_tags": _("Tags"),
             "search_entities": _("Entities"),
-            "search_mute": _("Mute"),
             "set_account": _("Account"),
             "set_type": _("Type"),
             "set_is_paid": _("Paid"),
@@ -197,7 +189,6 @@ class UpdateOrCreateTransactionRuleActionForm(forms.ModelForm):
             "set_category": _("Category"),
             "set_internal_note": _("Internal Note"),
             "set_internal_id": _("Internal ID"),
-            "set_mute": _("Mute"),
         }
 
     def __init__(self, *args, **kwargs):
@@ -209,149 +200,138 @@ class UpdateOrCreateTransactionRuleActionForm(forms.ModelForm):
         self.helper.form_method = "post"
 
         self.helper.layout = Layout(
-            "order",
-            Accordion(
+            BS5Accordion(
                 AccordionGroup(
                     _("Search Criteria"),
                     Field("filter", rows=1),
                     Row(
                         Column(
                             Field("search_type_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_type", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_is_paid_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_is_paid", rows=1),
-                            css_class="col-span-12 md:col-span-8",
-                        ),
-                    ),
-                    Row(
-                        Column(
-                            Field("search_mute_operator"),
-                            css_class="col-span-12 md:col-span-4",
-                        ),
-                        Column(
-                            Field("search_mute", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_account_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_account", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_entities_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_entities", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_date_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_date", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_reference_date_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_reference_date", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_description_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_description", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_amount_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_amount", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_category_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_category", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_tags_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_tags", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_notes_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_notes", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_internal_note_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_internal_note", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     Row(
                         Column(
                             Field("search_internal_id_operator"),
-                            css_class="col-span-12 md:col-span-4",
+                            css_class="form-group col-md-4",
                         ),
                         Column(
                             Field("search_internal_id", rows=1),
-                            css_class="col-span-12 md:col-span-8",
+                            css_class="form-group col-md-8",
                         ),
                     ),
                     active=True,
@@ -360,7 +340,6 @@ class UpdateOrCreateTransactionRuleActionForm(forms.ModelForm):
                     _("Set Values"),
                     Field("set_type", rows=1),
                     Field("set_is_paid", rows=1),
-                    Field("set_mute", rows=1),
                     Field("set_account", rows=1),
                     Field("set_entities", rows=1),
                     Field("set_date", rows=1),
@@ -382,13 +361,17 @@ class UpdateOrCreateTransactionRuleActionForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -398,106 +381,3 @@ class UpdateOrCreateTransactionRuleActionForm(forms.ModelForm):
         if commit:
             instance.save()
         return instance
-
-
-class DryRunCreatedTransacion(forms.Form):
-    transaction = DynamicModelChoiceField(
-        model=Transaction,
-        to_field_name="id",
-        label=_("Transaction"),
-        required=True,
-        queryset=Transaction.objects.none(),
-        widget=TransactionSelect(clear_button=False, income=True, expense=True),
-        help_text=_("Type to search for a transaction"),
-    )
-
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-
-        self.helper = FormHelper()
-        self.helper.form_tag = False
-        self.helper.layout = Layout(
-            "transaction",
-            FormActions(
-                NoClassSubmit("submit", _("Test"), css_class="btn btn-primary"),
-            ),
-        )
-
-        if self.data.get("transaction"):
-            try:
-                transaction = Transaction.objects.get(id=self.data.get("transaction"))
-            except Transaction.DoesNotExist:
-                transaction = None
-
-            if transaction:
-                self.fields["transaction"].queryset = Transaction.objects.filter(
-                    id=transaction.id
-                )
-
-
-class DryRunDeletedTransacion(forms.Form):
-    transaction = DynamicModelChoiceField(
-        model=Transaction,
-        to_field_name="id",
-        label=_("Transaction"),
-        required=True,
-        queryset=Transaction.objects.none(),
-        widget=TransactionSelect(clear_button=False, income=True, expense=True),
-        help_text=_("Type to search for a transaction"),
-    )
-
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-
-        self.helper = FormHelper()
-        self.helper.form_tag = False
-        self.helper.layout = Layout(
-            "transaction",
-            FormActions(
-                NoClassSubmit("submit", _("Test"), css_class="btn btn-primary"),
-            ),
-        )
-
-        if self.data.get("transaction"):
-            try:
-                transaction = Transaction.objects.get(id=self.data.get("transaction"))
-            except Transaction.DoesNotExist:
-                transaction = None
-
-            if transaction:
-                self.fields["transaction"].queryset = Transaction.objects.filter(
-                    id=transaction.id
-                )
-
-
-class DryRunUpdatedTransactionForm(BulkEditTransactionForm):
-    transaction = DynamicModelChoiceField(
-        model=Transaction,
-        to_field_name="id",
-        label=_("Transaction"),
-        required=True,
-        queryset=Transaction.objects.none(),
-        widget=TransactionSelect(clear_button=False, income=True, expense=True),
-        help_text=_("Type to search for a transaction"),
-    )
-
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        self.helper.layout.insert(0, "transaction")
-        self.helper.layout.insert(1, HTML('<hr class="hr my-3" />'))
-
-        # Change submit button
-        self.helper.layout[-1] = FormActions(
-            NoClassSubmit("submit", _("Test"), css_class="btn btn-primary")
-        )
-
-        if self.data.get("transaction"):
-            try:
-                transaction = Transaction.objects.get(id=self.data.get("transaction"))
-            except Transaction.DoesNotExist:
-                transaction = None
-
-            if transaction:
-                self.fields["transaction"].queryset = Transaction.objects.filter(
-                    id=transaction.id
-                )

app/apps/rules/migrations/0015_alter_transactionruleaction_options_and_more.py

@@ -1,39 +0,0 @@
-# Generated by Django 5.2 on 2025-08-30 18:48
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ("rules", "0014_alter_transactionrule_owner_and_more"),
-    ]
-
-    operations = [
-        migrations.AlterModelOptions(
-            name="transactionruleaction",
-            options={
-                "ordering": ["order"],
-                "verbose_name": "Edit transaction action",
-                "verbose_name_plural": "Edit transaction actions",
-            },
-        ),
-        migrations.AlterModelOptions(
-            name="updateorcreatetransactionruleaction",
-            options={
-                "ordering": ["order"],
-                "verbose_name": "Update or create transaction action",
-                "verbose_name_plural": "Update or create transaction actions",
-            },
-        ),
-        migrations.AddField(
-            model_name="transactionruleaction",
-            name="order",
-            field=models.PositiveIntegerField(default=0, verbose_name="Order"),
-        ),
-        migrations.AddField(
-            model_name="updateorcreatetransactionruleaction",
-            name="order",
-            field=models.PositiveIntegerField(default=0, verbose_name="Order"),
-        ),
-    ]

app/apps/rules/migrations/0016_transactionrule_sequenced.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.2.5 on 2025-08-31 18:42
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('rules', '0015_alter_transactionruleaction_options_and_more'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='transactionrule',
-            name='sequenced',
-            field=models.BooleanField(default=False, verbose_name='Sequenced'),
-        ),
-    ]

app/apps/rules/migrations/0017_updateorcreatetransactionruleaction_search_mute_and_more.py

@@ -1,33 +0,0 @@
-# Generated by Django 5.2.5 on 2025-08-31 19:09
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('rules', '0016_transactionrule_sequenced'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='updateorcreatetransactionruleaction',
-            name='search_mute',
-            field=models.TextField(blank=True, verbose_name='Search Mute'),
-        ),
-        migrations.AddField(
-            model_name='updateorcreatetransactionruleaction',
-            name='search_mute_operator',
-            field=models.CharField(choices=[('exact', 'is exactly'), ('contains', 'contains'), ('startswith', 'starts with'), ('endswith', 'ends with'), ('eq', 'equals'), ('gt', 'greater than'), ('lt', 'less than'), ('gte', 'greater than or equal'), ('lte', 'less than or equal')], default='exact', max_length=10, verbose_name='Mute Operator'),
-        ),
-        migrations.AddField(
-            model_name='updateorcreatetransactionruleaction',
-            name='set_mute',
-            field=models.TextField(blank=True, verbose_name='Mute'),
-        ),
-        migrations.AlterField(
-            model_name='transactionruleaction',
-            name='field',
-            field=models.CharField(choices=[('account', 'Account'), ('type', 'Type'), ('is_paid', 'Paid'), ('date', 'Date'), ('reference_date', 'Reference Date'), ('mute', 'Mute'), ('amount', 'Amount'), ('description', 'Description'), ('notes', 'Notes'), ('category', 'Category'), ('tags', 'Tags'), ('entities', 'Entities'), ('internal_nome', 'Internal Note'), ('internal_id', 'Internal ID')], max_length=50, verbose_name='Field'),
-        ),
-    ]

app/apps/rules/migrations/0018_transactionrule_order.py

@@ -1,18 +0,0 @@
-# Generated by Django 5.2.5 on 2025-09-02 14:52
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('rules', '0017_updateorcreatetransactionruleaction_search_mute_and_more'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='transactionrule',
-            name='order',
-            field=models.PositiveIntegerField(default=0, verbose_name='Order'),
-        ),
-    ]

app/apps/rules/models.py

@@ -13,11 +13,6 @@ class TransactionRule(SharedObject):
     name = models.CharField(max_length=100, verbose_name=_("Name"))
     description = models.TextField(blank=True, null=True, verbose_name=_("Description"))
     trigger = models.TextField(verbose_name=_("Trigger"))
-    sequenced = models.BooleanField(
-        verbose_name=_("Sequenced"),
-        default=False,
-    )
-    order = models.PositiveIntegerField(default=0, verbose_name=_("Order"))
 
     objects = SharedObjectManager()
     all_objects = models.Manager()  # Unfiltered manager
@@ -37,15 +32,12 @@ class TransactionRuleAction(models.Model):
         is_paid = "is_paid", _("Paid")
         date = "date", _("Date")
         reference_date = "reference_date", _("Reference Date")
-        mute = "mute", _("Mute")
         amount = "amount", _("Amount")
         description = "description", _("Description")
         notes = "notes", _("Notes")
         category = "category", _("Category")
         tags = "tags", _("Tags")
         entities = "entities", _("Entities")
-        internal_note = "internal_nome", _("Internal Note")
-        internal_id = "internal_id", _("Internal ID")
 
     rule = models.ForeignKey(
         TransactionRule,
@@ -59,7 +51,6 @@ class TransactionRuleAction(models.Model):
         verbose_name=_("Field"),
     )
     value = models.TextField(verbose_name=_("Value"))
-    order = models.PositiveIntegerField(default=0, verbose_name=_("Order"))
 
     def __str__(self):
         return f"{self.rule} - {self.field} - {self.value}"
@@ -68,11 +59,6 @@ class TransactionRuleAction(models.Model):
         verbose_name = _("Edit transaction action")
         verbose_name_plural = _("Edit transaction actions")
         unique_together = (("rule", "field"),)
-        ordering = ["order"]
-
-    @property
-    def action_type(self):
-        return "edit_transaction"
 
 
 class UpdateOrCreateTransactionRuleAction(models.Model):
@@ -251,17 +237,6 @@ class UpdateOrCreateTransactionRuleAction(models.Model):
         verbose_name="Internal ID Operator",
     )
 
-    search_mute = models.TextField(
-        verbose_name="Search Mute",
-        blank=True,
-    )
-    search_mute_operator = models.CharField(
-        max_length=10,
-        choices=SearchOperator.choices,
-        default=SearchOperator.EXACT,
-        verbose_name="Mute Operator",
-    )
-
     # Set fields
     set_account = models.TextField(
         verbose_name=_("Account"),
@@ -315,21 +290,10 @@ class UpdateOrCreateTransactionRuleAction(models.Model):
         verbose_name=_("Tags"),
         blank=True,
     )
-    set_mute = models.TextField(
-        verbose_name=_("Mute"),
-        blank=True,
-    )
-
-    order = models.PositiveIntegerField(default=0, verbose_name=_("Order"))
 
     class Meta:
         verbose_name = _("Update or create transaction action")
         verbose_name_plural = _("Update or create transaction actions")
-        ordering = ["order"]
-
-    @property
-    def action_type(self):
-        return "update_or_create_transaction"
 
     def __str__(self):
         return f"Update or create transaction action for {self.rule}"
@@ -361,10 +325,6 @@ class UpdateOrCreateTransactionRuleAction(models.Model):
             value = simple.eval(self.search_is_paid)
             search_query &= add_to_query("is_paid", value, self.search_is_paid_operator)
 
-        if self.search_mute:
-            value = simple.eval(self.search_mute)
-            search_query &= add_to_query("mute", value, self.search_mute_operator)
-
         if self.search_date:
             value = simple.eval(self.search_date)
             search_query &= add_to_query("date", value, self.search_date_operator)

app/apps/rules/signals.py

@@ -9,17 +9,40 @@ from apps.transactions.models import (
 )
 from apps.rules.tasks import check_for_transaction_rules
 from apps.common.middleware.thread_local import get_current_user
-from apps.rules.utils.transactions import serialize_transaction
 
 
 @receiver(transaction_created)
 @receiver(transaction_updated)
 @receiver(transaction_deleted)
 def transaction_changed_receiver(sender: Transaction, signal, **kwargs):
-    old_data = kwargs.get("old_data")
     if signal is transaction_deleted:
         # Serialize transaction data for processing
-        transaction_data = serialize_transaction(sender, deleted=True)
+        transaction_data = {
+            "id": sender.id,
+            "account": (sender.account.id, sender.account.name),
+            "account_group": (
+                sender.account.group.id if sender.account.group else None,
+                sender.account.group.name if sender.account.group else None,
+            ),
+            "type": str(sender.type),
+            "is_paid": sender.is_paid,
+            "is_asset": sender.account.is_asset,
+            "is_archived": sender.account.is_archived,
+            "category": (
+                sender.category.id if sender.category else None,
+                sender.category.name if sender.category else None,
+            ),
+            "date": sender.date.isoformat(),
+            "reference_date": sender.reference_date.isoformat(),
+            "amount": str(sender.amount),
+            "description": sender.description,
+            "notes": sender.notes,
+            "tags": list(sender.tags.values_list("id", "name")),
+            "entities": list(sender.entities.values_list("id", "name")),
+            "deleted": True,
+            "internal_note": sender.internal_note,
+            "internal_id": sender.internal_id,
+        }
 
         check_for_transaction_rules.defer(
             transaction_data=transaction_data,
@@ -36,9 +59,6 @@ def transaction_changed_receiver(sender: Transaction, signal, **kwargs):
         dca_entry.amount_received = sender.amount
         dca_entry.save()
 
-    if signal is transaction_updated and old_data:
-        old_data = serialize_transaction(old_data, deleted=False)
-
     check_for_transaction_rules.defer(
         instance_id=sender.id,
         user_id=get_current_user().id,
@@ -47,5 +67,4 @@ def transaction_changed_receiver(sender: Transaction, signal, **kwargs):
             if signal is transaction_created
             else "transaction_updated"
         ),
-        old_data=old_data,
     )

app/apps/rules/urls.py

@@ -42,21 +42,6 @@ urlpatterns = [
         views.transaction_rule_take_ownership,
         name="transaction_rule_take_ownership",
     ),
-    path(
-        "rules/transaction/<int:pk>/dry-run/created/",
-        views.dry_run_rule_created,
-        name="transaction_rule_dry_run_created",
-    ),
-    path(
-        "rules/transaction/<int:pk>/dry-run/deleted/",
-        views.dry_run_rule_deleted,
-        name="transaction_rule_dry_run_deleted",
-    ),
-    path(
-        "rules/transaction/<int:pk>/dry-run/updated/",
-        views.dry_run_rule_updated,
-        name="transaction_rule_dry_run_updated",
-    ),
     path(
         "rules/transaction/<int:pk>/share/",
         views.transaction_rule_share,

app/apps/rules/utils/transactions.py

@@ -1,101 +0,0 @@
-import logging
-from decimal import Decimal
-
-from django.db.models import Sum, Value, DecimalField, Case, When, F
-from django.db.models.functions import Coalesce
-
-from apps.transactions.models import (
-    Transaction,
-)
-
-logger = logging.getLogger(__name__)
-
-
-class TransactionsGetter:
-    def __init__(self, **filters):
-        self.__queryset = Transaction.objects.filter(**filters)
-
-    def exclude(self, **exclude_filters):
-        self.__queryset = self.__queryset.exclude(**exclude_filters)
-
-        return self
-
-    @property
-    def sum(self):
-        return self.__queryset.aggregate(
-            total=Coalesce(
-                Sum("amount"), Value(Decimal("0")), output_field=DecimalField()
-            )
-        )["total"]
-
-    @property
-    def balance(self):
-        return abs(
-            self.__queryset.aggregate(
-                balance=Coalesce(
-                    Sum(
-                        Case(
-                            When(type=Transaction.Type.EXPENSE, then=-F("amount")),
-                            default=F("amount"),
-                            output_field=DecimalField(),
-                        )
-                    ),
-                    Value(Decimal("0")),
-                    output_field=DecimalField(),
-                )
-            )["balance"]
-        )
-
-    @property
-    def raw_balance(self):
-        return self.__queryset.aggregate(
-            balance=Coalesce(
-                Sum(
-                    Case(
-                        When(type=Transaction.Type.EXPENSE, then=-F("amount")),
-                        default=F("amount"),
-                        output_field=DecimalField(),
-                    )
-                ),
-                Value(Decimal("0")),
-                output_field=DecimalField(),
-            )
-        )["balance"]
-
-
-def serialize_transaction(sender: Transaction, deleted: bool):
-    return {
-        "id": sender.id,
-        "account": (sender.account.id, sender.account.name),
-        "account_group": (
-            sender.account.group.id if sender.account.group else None,
-            sender.account.group.name if sender.account.group else None,
-        ),
-        "type": str(sender.type),
-        "is_paid": sender.is_paid,
-        "is_asset": sender.account.is_asset,
-        "is_archived": sender.account.is_archived,
-        "category": (
-            sender.category.id if sender.category else None,
-            sender.category.name if sender.category else None,
-        ),
-        "date": sender.date.isoformat(),
-        "reference_date": sender.reference_date.isoformat(),
-        "amount": str(sender.amount),
-        "description": sender.description,
-        "notes": sender.notes,
-        "tags": list(sender.tags.values_list("id", "name")),
-        "entities": list(sender.entities.values_list("id", "name")),
-        "deleted": deleted,
-        "internal_note": sender.internal_note,
-        "internal_id": sender.internal_id,
-        "mute": sender.mute,
-        "installment_id": sender.installment_id if sender.installment_plan else None,
-        "installment_total": (
-            sender.installment_plan.number_of_installments
-            if sender.installment_plan is not None
-            else None
-        ),
-        "installment": sender.installment_plan is not None,
-        "recurring_transaction": sender.recurring_transaction is not None,
-    }

app/apps/rules/views.py

@@ -1,10 +1,5 @@
-from itertools import chain
-
-from copy import deepcopy
-
 from django.contrib import messages
 from django.contrib.auth.decorators import login_required
-from django.db import transaction
 from django.http import HttpResponse
 from django.shortcuts import render, get_object_or_404, redirect
 from django.utils.translation import gettext_lazy as _
@@ -15,9 +10,6 @@ from apps.rules.forms import (
     TransactionRuleForm,
     TransactionRuleActionForm,
     UpdateOrCreateTransactionRuleActionForm,
-    DryRunCreatedTransacion,
-    DryRunDeletedTransacion,
-    DryRunUpdatedTransactionForm,
 )
 from apps.rules.models import (
     TransactionRule,
@@ -27,11 +19,6 @@ from apps.rules.models import (
 from apps.common.models import SharedObject
 from apps.common.forms import SharedObjectForm
 from apps.common.decorators.demo import disabled_on_demo
-from apps.rules.tasks import check_for_transaction_rules
-from apps.common.middleware.thread_local import get_current_user
-from apps.rules.signals import transaction_created, transaction_updated
-from apps.rules.utils.transactions import serialize_transaction
-from apps.transactions.models import Transaction
 
 
 @login_required
@@ -49,7 +36,7 @@ def rules_index(request):
 @disabled_on_demo
 @require_http_methods(["GET"])
 def rules_list(request):
-    transaction_rules = TransactionRule.objects.all().order_by("order", "id")
+    transaction_rules = TransactionRule.objects.all().order_by("id")
     return render(
         request,
         "rules/fragments/list.html",
@@ -153,20 +140,10 @@ def transaction_rule_edit(request, transaction_rule_id):
 def transaction_rule_view(request, transaction_rule_id):
     transaction_rule = get_object_or_404(TransactionRule, id=transaction_rule_id)
 
-    edit_actions = transaction_rule.transaction_actions.all()
-    update_or_create_actions = (
-        transaction_rule.update_or_create_transaction_actions.all()
-    )
-
-    all_actions = sorted(
-        chain(edit_actions, update_or_create_actions),
-        key=lambda a: a.order,
-    )
-
     return render(
         request,
         "rules/fragments/transaction_rule/view.html",
-        {"transaction_rule": transaction_rule, "all_actions": all_actions},
+        {"transaction_rule": transaction_rule},
     )
 
 
@@ -429,156 +406,3 @@ def update_or_create_transaction_rule_action_delete(request, pk):
             "HX-Trigger": "updated, hide_offcanvas",
         },
     )
-
-
-@only_htmx
-@login_required
-@disabled_on_demo
-@require_http_methods(["GET", "POST"])
-def dry_run_rule_created(request, pk):
-    rule = get_object_or_404(TransactionRule, id=pk)
-    logs = None
-    results = None
-
-    if request.method == "POST":
-        form = DryRunCreatedTransacion(request.POST)
-        if form.is_valid():
-            try:
-                with transaction.atomic():
-                    logs, results = check_for_transaction_rules(
-                        instance_id=form.cleaned_data["transaction"].id,
-                        signal="transaction_created",
-                        dry_run=True,
-                        rule_id=rule.id,
-                        user_id=get_current_user().id,
-                    )
-                    logs = "\n".join(logs)
-
-                    response = render(
-                        request,
-                        "rules/fragments/transaction_rule/dry_run/created.html",
-                        {"form": form, "rule": rule, "logs": logs, "results": results},
-                    )
-
-                    raise Exception("ROLLBACK")
-            except Exception:
-                pass
-
-            return response
-
-    else:
-        form = DryRunCreatedTransacion()
-
-    return render(
-        request,
-        "rules/fragments/transaction_rule/dry_run/created.html",
-        {"form": form, "rule": rule, "logs": logs, "results": results},
-    )
-
-
-@only_htmx
-@login_required
-@disabled_on_demo
-@require_http_methods(["GET", "POST"])
-def dry_run_rule_deleted(request, pk):
-    rule = get_object_or_404(TransactionRule, id=pk)
-    logs = None
-    results = None
-
-    if request.method == "POST":
-        form = DryRunDeletedTransacion(request.POST)
-        if form.is_valid():
-            try:
-                with transaction.atomic():
-                    logs, results = check_for_transaction_rules(
-                        instance_id=form.cleaned_data["transaction"].id,
-                        signal="transaction_deleted",
-                        dry_run=True,
-                        rule_id=rule.id,
-                        user_id=get_current_user().id,
-                    )
-                    logs = "\n".join(logs)
-
-                    response = render(
-                        request,
-                        "rules/fragments/transaction_rule/dry_run/created.html",
-                        {"form": form, "rule": rule, "logs": logs, "results": results},
-                    )
-
-                    raise Exception("ROLLBACK")
-            except Exception:
-                pass
-
-        return response
-
-    else:
-        form = DryRunDeletedTransacion()
-
-    return render(
-        request,
-        "rules/fragments/transaction_rule/dry_run/deleted.html",
-        {"form": form, "rule": rule, "logs": logs, "results": results},
-    )
-
-
-@only_htmx
-@login_required
-@disabled_on_demo
-@require_http_methods(["GET", "POST"])
-def dry_run_rule_updated(request, pk):
-    rule = get_object_or_404(TransactionRule, id=pk)
-    logs = None
-    results = None
-
-    if request.method == "POST":
-        form = DryRunUpdatedTransactionForm(request.POST)
-        if form.is_valid():
-            base_transaction = Transaction.objects.get(
-                id=request.POST.get("transaction")
-            )
-            old_data = deepcopy(base_transaction)
-            try:
-                with transaction.atomic():
-                    for field_name, value in form.cleaned_data.items():
-                        if value or isinstance(
-                            value, bool
-                        ):  # Only update fields that have been filled in the form
-                            if field_name == "tags":
-                                base_transaction.tags.set(value)
-                            elif field_name == "entities":
-                                base_transaction.entities.set(value)
-                            else:
-                                setattr(base_transaction, field_name, value)
-
-                    base_transaction.save()
-
-                    logs, results = check_for_transaction_rules(
-                        instance_id=base_transaction.id,
-                        signal="transaction_updated",
-                        dry_run=True,
-                        rule_id=rule.id,
-                        user_id=get_current_user().id,
-                        old_data=old_data,
-                    )
-                    logs = "\n".join(logs) if logs else ""
-
-                    response = render(
-                        request,
-                        "rules/fragments/transaction_rule/dry_run/updated.html",
-                        {"form": form, "rule": rule, "logs": logs, "results": results},
-                    )
-
-                    # This will rollback the transaction
-                    raise Exception("ROLLBACK")
-            except Exception:
-                pass
-
-            return response
-    else:
-        form = DryRunUpdatedTransactionForm(initial={"is_paid": None, "type": None})
-
-    return render(
-        request,
-        "rules/fragments/transaction_rule/dry_run/updated.html",
-        {"form": form, "rule": rule, "logs": logs, "results": results},
-    )

app/apps/transactions/filters.py

@@ -1,4 +1,11 @@
 import django_filters
+from crispy_forms.helper import FormHelper
+from crispy_forms.layout import Layout, Field, Row, Column
+from django import forms
+from django.db.models import Q
+from django.utils.translation import gettext_lazy as _
+from django_filters import Filter
+
 from apps.accounts.models import Account
 from apps.common.fields.month_year import MonthYearFormField
 from apps.common.widgets.datepicker import AirDatePickerInput
@@ -8,15 +15,9 @@ from apps.currencies.models import Currency
 from apps.transactions.models import (
     Transaction,
     TransactionCategory,
-    TransactionEntity,
     TransactionTag,
+    TransactionEntity,
 )
-from crispy_forms.helper import FormHelper
-from crispy_forms.layout import Column, Field, Layout, Row
-from django import forms
-from django.db.models import Q
-from django.utils.translation import gettext_lazy as _
-from django_filters import Filter
 
 SITUACAO_CHOICES = (
     ("1", _("Paid")),
@@ -59,20 +60,26 @@ class TransactionsFilter(django_filters.FilterSet):
         label=_("Currencies"),
         widget=TomSelectMultiple(checkboxes=True, remove_button=True),
     )
-    category = django_filters.MultipleChoiceFilter(
+    category = django_filters.ModelMultipleChoiceFilter(
+        field_name="category__name",
+        queryset=TransactionCategory.objects.all(),
+        to_field_name="name",
         label=_("Categories"),
         widget=TomSelectMultiple(checkboxes=True, remove_button=True),
-        method="filter_category",
     )
-    tags = django_filters.MultipleChoiceFilter(
+    tags = django_filters.ModelMultipleChoiceFilter(
+        field_name="tags__name",
+        queryset=TransactionTag.objects.all(),
+        to_field_name="name",
         label=_("Tags"),
         widget=TomSelectMultiple(checkboxes=True, remove_button=True),
-        method="filter_tags",
     )
-    entities = django_filters.MultipleChoiceFilter(
+    entities = django_filters.ModelMultipleChoiceFilter(
+        field_name="entities__name",
+        queryset=TransactionEntity.objects.all(),
+        to_field_name="name",
         label=_("Entities"),
         widget=TomSelectMultiple(checkboxes=True, remove_button=True),
-        method="filter_entities",
     )
     is_paid = django_filters.MultipleChoiceFilter(
         choices=SITUACAO_CHOICES,
@@ -118,7 +125,6 @@ class TransactionsFilter(django_filters.FilterSet):
             "is_paid",
             "category",
             "tags",
-            "entities",
             "date_start",
             "date_end",
             "reference_date_start",
@@ -158,12 +164,14 @@ class TransactionsFilter(django_filters.FilterSet):
             Field("description"),
             Row(Column("date_start"), Column("date_end")),
             Row(
-                Column("reference_date_start"),
-                Column("reference_date_end"),
+                Column("reference_date_start", css_class="form-group col-md-6 mb-0"),
+                Column("reference_date_end", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             Row(
-                Column("from_amount"),
-                Column("to_amount"),
+                Column("from_amount", css_class="form-group col-md-6 mb-0"),
+                Column("to_amount", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             Field("account", size=1),
             Field("currency", size=1),
@@ -178,93 +186,6 @@ class TransactionsFilter(django_filters.FilterSet):
         self.form.fields["date_end"].widget = AirDatePickerInput()
 
         self.form.fields["account"].queryset = Account.objects.all()
-        category_choices = list(
-            TransactionCategory.objects.values_list("name", "name").order_by("name")
-        )
-        custom_choices = [
-            ("any", _("Categorized")),
-            ("uncategorized", _("Uncategorized")),
-        ]
-        self.form.fields["category"].choices = custom_choices + category_choices
-        tag_choices = list(
-            TransactionTag.objects.values_list("name", "name").order_by("name")
-        )
-        custom_tag_choices = [("any", _("Tagged")), ("untagged", _("Untagged"))]
-        self.form.fields["tags"].choices = custom_tag_choices + tag_choices
-        entity_choices = list(
-            TransactionEntity.objects.values_list("name", "name").order_by("name")
-        )
-        custom_entity_choices = [
-            ("any", _("Any entity")),
-            ("no_entity", _("No entity")),
-        ]
-        self.form.fields["entities"].choices = custom_entity_choices + entity_choices
-
-    @staticmethod
-    def filter_category(queryset, name, value):
-        if not value:
-            return queryset
-
-        value = list(value)
-
-        if "any" in value:
-            return queryset.filter(category__isnull=False)
-
-        q = Q()
-        if "uncategorized" in value:
-            q |= Q(category__isnull=True)
-            value.remove("uncategorized")
-
-        if value:
-            q |= Q(category__name__in=value)
-
-        if q.children:
-            return queryset.filter(q)
-
-        return queryset
-
-    @staticmethod
-    def filter_tags(queryset, name, value):
-        if not value:
-            return queryset
-
-        value = list(value)
-
-        if "any" in value:
-            return queryset.filter(tags__isnull=False).distinct()
-
-        q = Q()
-        if "untagged" in value:
-            q |= Q(tags__isnull=True)
-            value.remove("untagged")
-
-        if value:
-            q |= Q(tags__name__in=value)
-
-        if q.children:
-            return queryset.filter(q).distinct()
-
-        return queryset
-
-    @staticmethod
-    def filter_entities(queryset, name, value):
-        if not value:
-            return queryset
-
-        value = list(value)
-
-        if "any" in value:
-            return queryset.filter(entities__isnull=False).distinct()
-
-        q = Q()
-        if "no_entity" in value:
-            q |= Q(entities__isnull=True)
-            value.remove("no_entity")
-
-        if value:
-            q |= Q(entities__name__in=value)
-
-        if q.children:
-            return queryset.filter(q).distinct()
-
-        return queryset
+        self.form.fields["category"].queryset = TransactionCategory.objects.all()
+        self.form.fields["tags"].queryset = TransactionTag.objects.all()
+        self.form.fields["entities"].queryset = TransactionEntity.objects.all()

app/apps/transactions/forms.py

@@ -1,38 +1,37 @@
-from copy import deepcopy
+from crispy_bootstrap5.bootstrap5 import Switch, BS5Accordion
+from crispy_forms.bootstrap import FormActions, AccordionGroup, AppendedText
+from crispy_forms.helper import FormHelper
+from crispy_forms.layout import (
+    Layout,
+    Row,
+    Column,
+    Field,
+    Div,
+    HTML,
+)
+from django import forms
+from django.db.models import Q
+from django.utils.translation import gettext_lazy as _
 
 from apps.accounts.models import Account
 from apps.common.fields.forms.dynamic_select import (
     DynamicModelChoiceField,
     DynamicModelMultipleChoiceField,
 )
-from apps.common.widgets.crispy.daisyui import Switch
 from apps.common.widgets.crispy.submit import NoClassSubmit
 from apps.common.widgets.datepicker import AirDatePickerInput, AirMonthYearPickerInput
 from apps.common.widgets.decimal import ArbitraryDecimalDisplayNumberInput
 from apps.common.widgets.tom_select import TomSelect
 from apps.rules.signals import transaction_created, transaction_updated
 from apps.transactions.models import (
-    InstallmentPlan,
-    QuickTransaction,
-    RecurringTransaction,
     Transaction,
     TransactionCategory,
-    TransactionEntity,
     TransactionTag,
+    InstallmentPlan,
+    RecurringTransaction,
+    TransactionEntity,
+    QuickTransaction,
 )
-from crispy_forms.bootstrap import AccordionGroup, AppendedText, FormActions, Accordion
-from crispy_forms.helper import FormHelper
-from crispy_forms.layout import (
-    HTML,
-    Column,
-    Div,
-    Field,
-    Layout,
-    Row,
-)
-from django import forms
-from django.db.models import Q
-from django.utils.translation import gettext_lazy as _
 
 
 class TransactionForm(forms.ModelForm):
@@ -133,18 +132,21 @@ class TransactionForm(forms.ModelForm):
             ),
             Field("is_paid", template="transactions/widgets/paid_toggle_button.html"),
             Row(
-                Column("account"),
-                Column("entities"),
+                Column("account", css_class="form-group col-md-6 mb-0"),
+                Column("entities", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             Row(
-                Column(Field("date")),
-                Column(Field("reference_date")),
+                Column(Field("date"), css_class="form-group col-md-6 mb-0"),
+                Column(Field("reference_date"), css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             "description",
             Field("amount", inputmode="decimal"),
             Row(
-                Column("category"),
-                Column("tags"),
+                Column("category", css_class="form-group col-md-6 mb-0"),
+                Column("tags", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             "notes",
         )
@@ -160,18 +162,20 @@ class TransactionForm(forms.ModelForm):
             Field("is_paid", template="transactions/widgets/paid_toggle_button.html"),
             "account",
             Row(
-                Column(Field("date")),
-                Column(Field("reference_date")),
+                Column(Field("date"), css_class="form-group col-md-6 mb-0"),
+                Column(Field("reference_date"), css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             "description",
             Field("amount", inputmode="decimal"),
-            Accordion(
+            BS5Accordion(
                 AccordionGroup(
                     _("More"),
                     "entities",
                     Row(
-                        Column("category"),
-                        Column("tags"),
+                        Column("category", css_class="form-group col-md-6 mb-0"),
+                        Column("tags", css_class="form-group col-md-6 mb-0"),
+                        css_class="form-row",
                     ),
                     "notes",
                     active=False,
@@ -181,7 +185,9 @@ class TransactionForm(forms.ModelForm):
                 css_class="mb-3",
             ),
             FormActions(
-                NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                NoClassSubmit(
+                    "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                ),
             ),
         )
 
@@ -194,25 +200,29 @@ class TransactionForm(forms.ModelForm):
             )
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.fields["amount"].widget = ArbitraryDecimalDisplayNumberInput()
             self.helper.layout.append(
                 Div(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary"
+                    ),
                     NoClassSubmit(
                         "submit_and_similar",
                         _("Save and add similar"),
-                        css_class="btn btn-primary btn-soft",
+                        css_class="btn btn-outline-primary",
                     ),
                     NoClassSubmit(
                         "submit_and_another",
                         _("Save and add another"),
-                        css_class="btn btn-primary btn-soft",
+                        css_class="btn btn-outline-primary",
                     ),
-                    css_class="flex flex-col gap-2 mt-3",
+                    css_class="d-grid gap-2",
                 ),
             )
 
@@ -229,16 +239,11 @@ class TransactionForm(forms.ModelForm):
     def save(self, **kwargs):
         is_new = not self.instance.id
 
-        if not is_new:
-            old_data = deepcopy(Transaction.objects.get(pk=self.instance.id))
-        else:
-            old_data = None
-
         instance = super().save(**kwargs)
         if is_new:
             transaction_created.send(sender=instance)
         else:
-            transaction_updated.send(sender=instance, old_data=old_data)
+            transaction_updated.send(sender=instance)
 
         return instance
 
@@ -336,16 +341,23 @@ class QuickTransactionForm(forms.ModelForm):
             ),
             Field("is_paid", template="transactions/widgets/paid_toggle_button.html"),
             "name",
-            HTML('<hr class="hr my-3" />'),
+            HTML("<hr />"),
             Row(
-                Column("account"),
-                Column("entities"),
+                Column("account", css_class="form-group col-md-6 mb-0"),
+                Column("entities", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
+            ),
+            Row(
+                Column(Field("date"), css_class="form-group col-md-6 mb-0"),
+                Column(Field("reference_date"), css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             "description",
             Field("amount", inputmode="decimal"),
             Row(
-                Column("category"),
-                Column("tags"),
+                Column("category", css_class="form-group col-md-6 mb-0"),
+                Column("tags", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             "notes",
             Switch("mute"),
@@ -358,131 +370,58 @@ class QuickTransactionForm(forms.ModelForm):
             )
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.fields["amount"].widget = ArbitraryDecimalDisplayNumberInput()
             self.helper.layout.append(
-                FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                Div(
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary"
+                    ),
+                    css_class="d-grid gap-2",
                 ),
             )
 
 
-class BulkEditTransactionForm(forms.Form):
-    type = forms.ChoiceField(
-        choices=(Transaction.Type.choices),
-        required=False,
-        label=_("Type"),
-    )
-    is_paid = forms.NullBooleanField(
-        required=False,
-        label=_("Paid"),
-    )
-    account = DynamicModelChoiceField(
-        model=Account,
-        required=False,
-        label=_("Account"),
-        queryset=Account.objects.filter(is_archived=False),
-        widget=TomSelect(clear_button=False, group_by="group"),
-    )
-    date = forms.DateField(
-        label=_("Date"),
-        required=False,
-        widget=AirDatePickerInput(clear_button=False),
-    )
-    reference_date = forms.DateField(
-        widget=AirMonthYearPickerInput(),
-        label=_("Reference Date"),
-        required=False,
-    )
-    amount = forms.DecimalField(
-        max_digits=42,
-        decimal_places=30,
-        required=False,
-        label=_("Amount"),
-        widget=ArbitraryDecimalDisplayNumberInput(),
-    )
-    description = forms.CharField(
-        max_length=500, required=False, label=_("Description")
-    )
-    notes = forms.CharField(
-        required=False,
-        widget=forms.Textarea(attrs={"rows": 3}),
-        label=_("Notes"),
-    )
-    category = DynamicModelChoiceField(
-        create_field="name",
-        model=TransactionCategory,
-        required=False,
-        label=_("Category"),
-        queryset=TransactionCategory.objects.filter(active=True),
-    )
-    tags = DynamicModelMultipleChoiceField(
-        model=TransactionTag,
-        to_field_name="name",
-        create_field="name",
-        required=False,
-        label=_("Tags"),
-        queryset=TransactionTag.objects.filter(active=True),
-    )
-    entities = DynamicModelMultipleChoiceField(
-        model=TransactionEntity,
-        to_field_name="name",
-        create_field="name",
-        required=False,
-        label=_("Entities"),
-        queryset=TransactionEntity.objects.all(),
-    )
+class BulkEditTransactionForm(TransactionForm):
+    is_paid = forms.NullBooleanField(required=False)
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
+        # Make all fields optional
+        for field_name, field in self.fields.items():
+            field.required = False
 
-        self.fields["account"].queryset = Account.objects.filter(
-            is_archived=False,
-        )
+        del self.helper.layout[-1]  # Remove button
+        del self.helper.layout[0:2]  # Remove type, is_paid field
 
-        self.fields["category"].queryset = TransactionCategory.objects.filter(
-            active=True
-        )
-        self.fields["tags"].queryset = TransactionTag.objects.filter(active=True)
-        self.fields["entities"].queryset = TransactionEntity.objects.all()
-
-        self.helper = FormHelper()
-        self.helper.form_tag = False
-        self.helper.form_method = "post"
-        self.helper.layout = Layout(
+        self.helper.layout.insert(
+            0,
             Field(
                 "type",
                 template="transactions/widgets/unselectable_income_expense_toggle_buttons.html",
             ),
+        )
+
+        self.helper.layout.insert(
+            1,
             Field(
                 "is_paid",
                 template="transactions/widgets/unselectable_paid_toggle_button.html",
             ),
-            Row(
-                Column("account"),
-                Column("entities"),
-            ),
-            Row(
-                Column(Field("date")),
-                Column(Field("reference_date")),
-            ),
-            "description",
-            Field("amount", inputmode="decimal"),
-            Row(
-                Column("category"),
-                Column("tags"),
-            ),
-            "notes",
-            FormActions(
-                NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
-            ),
         )
 
-        self.fields["amount"].widget = ArbitraryDecimalDisplayNumberInput()
-        self.fields["date"].widget = AirDatePickerInput(clear_button=False)
+        self.helper.layout.append(
+            FormActions(
+                NoClassSubmit(
+                    "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                ),
+            ),
+        )
 
 
 class TransferForm(forms.Form):
@@ -576,34 +515,62 @@ class TransferForm(forms.Form):
 
         self.helper.layout = Layout(
             Row(
-                Column(Field("date")),
+                Column(Field("date"), css_class="form-group col-md-6 mb-0"),
                 Column(
                     Field("reference_date"),
+                    css_class="form-group col-md-6 mb-0",
                 ),
+                css_class="form-row",
             ),
             Field("description"),
             Field("notes"),
             Switch("mute"),
             Row(
-                Column("from_account"),
-                Column(Field("from_amount")),
-                Column("from_category"),
-                Column("from_tags"),
-                css_class="bg-base-100 rounded-box p-4 border-base-content/60 border my-3",
+                Column(
+                    Row(
+                        Column(
+                            "from_account",
+                            css_class="form-group col-md-6 mb-0",
+                        ),
+                        Column(
+                            Field("from_amount"),
+                            css_class="form-group col-md-6 mb-0",
+                        ),
+                        css_class="form-row",
+                    ),
+                    Row(
+                        Column("from_category", css_class="form-group col-md-6 mb-0"),
+                        Column("from_tags", css_class="form-group col-md-6 mb-0"),
+                        css_class="form-row",
+                    ),
+                ),
+                css_class="p-1 mx-1 my-3 border rounded-3",
             ),
             Row(
                 Column(
-                    "to_account",
+                    Row(
+                        Column(
+                            "to_account",
+                            css_class="form-group col-md-6 mb-0",
+                        ),
+                        Column(
+                            Field("to_amount"),
+                            css_class="form-group col-md-6 mb-0",
+                        ),
+                        css_class="form-row",
+                    ),
+                    Row(
+                        Column("to_category", css_class="form-group col-md-6 mb-0"),
+                        Column("to_tags", css_class="form-group col-md-6 mb-0"),
+                        css_class="form-row",
+                    ),
                 ),
-                Column(
-                    Field("to_amount"),
-                ),
-                Column("to_category"),
-                Column("to_tags"),
-                css_class="bg-base-100 rounded-box p-4 border-base-content/60 border",
+                css_class="p-1 mx-1 my-3 border rounded-3",
             ),
             FormActions(
-                NoClassSubmit("submit", _("Transfer"), css_class="btn btn-primary"),
+                NoClassSubmit(
+                    "submit", _("Transfer"), css_class="btn btn-outline-primary w-100"
+                ),
             ),
         )
 
@@ -789,26 +756,30 @@ class InstallmentPlanForm(forms.ModelForm):
                 template="transactions/widgets/income_expense_toggle_buttons.html",
             ),
             Row(
-                Column("account"),
-                Column("entities"),
+                Column("account", css_class="form-group col-md-6 mb-0"),
+                Column("entities", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             "description",
             Switch("add_description_to_transaction"),
             "notes",
             Switch("add_notes_to_transaction"),
             Row(
-                Column("number_of_installments"),
-                Column("installment_start"),
+                Column("number_of_installments", css_class="form-group col-md-6 mb-0"),
+                Column("installment_start", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             Row(
-                Column("start_date", css_class="col-span-12 md:col-span-4"),
-                Column("reference_date", css_class="col-span-12 md:col-span-4"),
-                Column("recurrence", css_class="col-span-12 md:col-span-4"),
+                Column("start_date", css_class="form-group col-md-4 mb-0"),
+                Column("reference_date", css_class="form-group col-md-4 mb-0"),
+                Column("recurrence", css_class="form-group col-md-4 mb-0"),
+                css_class="form-row",
             ),
             "installment_amount",
             Row(
-                Column("category"),
-                Column("tags"),
+                Column("category", css_class="form-group col-md-6 mb-0"),
+                Column("tags", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
         )
 
@@ -818,13 +789,17 @@ class InstallmentPlanForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -857,13 +832,17 @@ class TransactionTagForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -885,13 +864,17 @@ class TransactionEntityForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -916,13 +899,17 @@ class TransactionCategoryForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -1031,26 +1018,30 @@ class RecurringTransactionForm(forms.ModelForm):
                 template="transactions/widgets/income_expense_toggle_buttons.html",
             ),
             Row(
-                Column("account"),
-                Column("entities"),
+                Column("account", css_class="form-group col-md-6 mb-0"),
+                Column("entities", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             "description",
             Switch("add_description_to_transaction"),
             "amount",
             Row(
-                Column("category"),
-                Column("tags"),
+                Column("category", css_class="form-group col-md-6 mb-0"),
+                Column("tags", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             "notes",
             Switch("add_notes_to_transaction"),
             Row(
-                Column("start_date"),
-                Column("reference_date"),
+                Column("start_date", css_class="form-group col-md-6 mb-0"),
+                Column("reference_date", css_class="form-group col-md-6 mb-0"),
+                css_class="form-row",
             ),
             Row(
-                Column("recurrence_interval", css_class="col-span-12 md:col-span-4"),
-                Column("recurrence_type", css_class="col-span-12 md:col-span-4"),
-                Column("end_date", css_class="col-span-12 md:col-span-4"),
+                Column("recurrence_interval", css_class="form-group col-md-4 mb-0"),
+                Column("recurrence_type", css_class="form-group col-md-4 mb-0"),
+                Column("end_date", css_class="form-group col-md-4 mb-0"),
+                css_class="form-row",
             ),
             AppendedText("keep_at_most", _("future transactions")),
         )
@@ -1062,13 +1053,17 @@ class RecurringTransactionForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -1090,6 +1085,5 @@ class RecurringTransactionForm(forms.ModelForm):
             instance.create_upcoming_transactions()
         else:
             instance.update_unpaid_transactions()
-            instance.generate_upcoming_transactions()
 
         return instance

app/apps/transactions/models.py

@@ -1,19 +1,5 @@
-import decimal
 import logging
-from copy import deepcopy
 
-from apps.common.fields.month_year import MonthYearModelField
-from apps.common.functions.decimals import truncate_decimal
-from apps.common.middleware.thread_local import get_current_user
-from apps.common.models import (
-    OwnedObject,
-    OwnedObjectManager,
-    SharedObject,
-    SharedObjectManager,
-)
-from apps.common.templatetags.decimal import drop_trailing_zeros, localize_number
-from apps.currencies.utils.convert import convert
-from apps.transactions.validators import validate_decimal_places, validate_non_negative
 from dateutil.relativedelta import relativedelta
 from django.conf import settings
 from django.core.validators import MinValueValidator
@@ -24,6 +10,19 @@ from django.template.defaultfilters import date
 from django.utils import timezone
 from django.utils.translation import gettext_lazy as _
 
+from apps.common.fields.month_year import MonthYearModelField
+from apps.common.functions.decimals import truncate_decimal
+from apps.common.templatetags.decimal import localize_number, drop_trailing_zeros
+from apps.currencies.utils.convert import convert
+from apps.transactions.validators import validate_decimal_places, validate_non_negative
+from apps.common.middleware.thread_local import get_current_user
+from apps.common.models import (
+    SharedObject,
+    SharedObjectManager,
+    OwnedObject,
+    OwnedObjectManager,
+)
+
 logger = logging.getLogger()
 
 
@@ -34,13 +33,13 @@ transaction_deleted = Signal()
 
 class SoftDeleteQuerySet(models.QuerySet):
     @staticmethod
-    def _emit_signals(instances, created=False, old_data=None):
+    def _emit_signals(instances, created=False):
         """Helper to emit signals for multiple instances"""
-        for i, instance in enumerate(instances):
+        for instance in instances:
             if created:
                 transaction_created.send(sender=instance)
             else:
-                transaction_updated.send(sender=instance, old_data=old_data[i])
+                transaction_updated.send(sender=instance)
 
     def bulk_create(self, objs, emit_signal=True, **kwargs):
         instances = super().bulk_create(objs, **kwargs)
@@ -51,25 +50,22 @@ class SoftDeleteQuerySet(models.QuerySet):
         return instances
 
     def bulk_update(self, objs, fields, emit_signal=True, **kwargs):
-        old_data = deepcopy(objs)
         result = super().bulk_update(objs, fields, **kwargs)
 
         if emit_signal:
-            self._emit_signals(objs, created=False, old_data=old_data)
+            self._emit_signals(objs, created=False)
 
         return result
 
     def update(self, emit_signal=True, **kwargs):
         # Get instances before update
         instances = list(self)
-        old_data = deepcopy(instances)
-
         result = super().update(**kwargs)
 
         if emit_signal:
             # Refresh instances to get new values
             refreshed = self.model.objects.filter(pk__in=[obj.pk for obj in instances])
-            self._emit_signals(refreshed, created=False, old_data=old_data)
+            self._emit_signals(refreshed, created=False)
 
         return result
 
@@ -380,39 +376,16 @@ class Transaction(OwnedObject):
         db_table = "transactions"
         default_manager_name = "objects"
 
-    def clean(self):
-        super().clean()
-
-        # Only process amount and reference_date if account exists
-        # If account is missing, Django's required field validation will handle it
-        try:
-            account = self.account
-        except Transaction.account.RelatedObjectDoesNotExist:
-            # Account doesn't exist, skip processing that depends on it
-            # Django will add the required field error
-            return
-
-        # Validate and normalize amount
-        if isinstance(self.amount, (str, int, float)):
-            self.amount = decimal.Decimal(str(self.amount))
-
+    def save(self, *args, **kwargs):
         self.amount = truncate_decimal(
-            value=self.amount, decimal_places=account.currency.decimal_places
+            value=self.amount, decimal_places=self.account.currency.decimal_places
         )
 
-        # Normalize reference_date
         if self.reference_date:
             self.reference_date = self.reference_date.replace(day=1)
         elif not self.reference_date and self.date:
             self.reference_date = self.date.replace(day=1)
 
-    def save(self, *args, **kwargs):
-        # This is here so Django validation doesn't trigger an error before clean() is ran
-        if not self.reference_date and self.date:
-            self.reference_date = self.date.replace(day=1)
-
-        # This is not recommended as it will run twice on some cases like form and API saves.
-        # We only do this here because we forgot to independently call it on multiple places.
         self.full_clean()
         super().save(*args, **kwargs)
 
@@ -470,58 +443,12 @@ class Transaction(OwnedObject):
         type_display = self.get_type_display()
         frmt_date = date(self.date, "SHORT_DATE_FORMAT")
         account = self.account
-        tags = (
-            ", ".join([x.name for x in self.tags.all()])
-            if self.id
-            else None or _("No tags")
-        )
+        tags = ", ".join([x.name for x in self.tags.all()]) or _("No tags")
         category = self.category or _("No category")
         amount = localize_number(drop_trailing_zeros(self.amount))
         description = self.description or _("No description")
         return f"[{frmt_date}][{type_display}][{account}] {description} • {category} • {tags} • {amount}"
 
-    def deepcopy(self, memo=None):
-        """
-        Creates a deep copy of the transaction instance.
-
-        This method returns a new, unsaved Transaction instance with the same
-        values as the original, including its many-to-many relationships.
-        The primary key and any other unique fields are reset to avoid
-        database integrity errors upon saving.
-        """
-        if memo is None:
-            memo = {}
-
-        # Create a new instance of the class
-        new_obj = self.__class__()
-        memo[id(self)] = new_obj
-
-        # Copy all concrete fields from the original to the new object
-        for field in self._meta.concrete_fields:
-            # Skip the primary key to allow the database to generate a new one
-            if field.primary_key:
-                continue
-
-            # Reset any unique fields to None to avoid constraint violations
-            if field.unique and field.name == "internal_id":
-                setattr(new_obj, field.name, None)
-                continue
-
-            # Copy the value of the field
-            setattr(new_obj, field.name, getattr(self, field.name))
-
-        # Save the new object to the database to get a primary key
-        new_obj.save()
-
-        # Copy the many-to-many relationships
-        for field in self._meta.many_to_many:
-            source_manager = getattr(self, field.name)
-            destination_manager = getattr(new_obj, field.name)
-            # Set the M2M relationships for the new object
-            destination_manager.set(source_manager.all())
-
-        return new_obj
-
 
 class InstallmentPlan(models.Model):
     class Recurrence(models.TextChoices):
@@ -870,8 +797,10 @@ class RecurringTransaction(models.Model):
             notes=self.notes if self.add_notes_to_transaction else "",
             owner=self.account.owner,
         )
-        created_transaction.tags.set(self.tags.all())
-        created_transaction.entities.set(self.entities.all())
+        if self.tags.exists():
+            created_transaction.tags.set(self.tags.all())
+        if self.entities.exists():
+            created_transaction.entities.set(self.entities.all())
 
     def get_recurrence_delta(self):
         if self.recurrence_type == self.RecurrenceType.DAY:

app/apps/transactions/tasks.py

@@ -13,9 +13,7 @@ logger = logging.getLogger(__name__)
 
 
 @app.periodic(cron="0 0 * * *")
-@app.task(
-    lock="generate_recurring_transactions", name="generate_recurring_transactions"
-)
+@app.task(name="generate_recurring_transactions")
 def generate_recurring_transactions(timestamp=None):
     try:
         RecurringTransaction.generate_upcoming_transactions()
@@ -28,7 +26,7 @@ def generate_recurring_transactions(timestamp=None):
 
 
 @app.periodic(cron="10 1 * * *")
-@app.task(lock="cleanup_deleted_transactions", name="cleanup_deleted_transactions")
+@app.task(name="cleanup_deleted_transactions")
 def cleanup_deleted_transactions(timestamp=None):
     if settings.ENABLE_SOFT_DELETE and settings.KEEP_DELETED_TRANSACTIONS_FOR == 0:
         return "KEEP_DELETED_TRANSACTIONS_FOR is 0, no cleanup performed."

app/apps/transactions/templatetags/currency_display.py

@@ -3,6 +3,7 @@ from decimal import Decimal
 from django import template
 from django.utils.formats import number_format
 
+
 register = template.Library()
 
 
@@ -12,27 +13,13 @@ def _format_string(prefix, amount, decimal_places, suffix):
             value=abs(amount), decimal_pos=decimal_places, force_grouping=True
         )
         if amount < 0:
-            return "-", prefix, formatted_amount, suffix
             return f"-{prefix}{formatted_amount}{suffix}"
         else:
-            return "", prefix, formatted_amount, suffix
             return f"{prefix}{formatted_amount}{suffix}"
     else:
-        return "", "", "ERR", ""
+        return "ERR"
 
 
 @register.simple_tag(name="currency_display")
-def currency_display(amount, prefix, suffix, decimal_places, string=False):
-    sign, prefix, amount, suffix = _format_string(
-        prefix, amount, decimal_places, suffix
-    )
-
-    if string:
-        return f"{sign}{prefix}{amount}{suffix}"
-
-    return {
-        "sign": sign,
-        "prefix": prefix,
-        "amount": amount,
-        "suffix": suffix,
-    }
+def currency_display(amount, prefix, suffix, decimal_places):
+    return _format_string(prefix, amount, decimal_places, suffix)

app/apps/transactions/tests.py

@@ -1,7 +1,9 @@
 import datetime
 from decimal import Decimal
+from datetime import date, timedelta
 
 from django.test import TestCase
+from django.core.exceptions import ValidationError
 from django.utils import timezone
 
 from apps.transactions.models import (
@@ -173,6 +175,6 @@ class RecurringTransactionTests(TestCase):
             recurrence_type=RecurringTransaction.RecurrenceType.MONTH,
             recurrence_interval=1,
         )
-        self.assertFalse(recurring.is_paused)
+        self.assertFalse(recurring.paused)
         self.assertEqual(recurring.recurrence_interval, 1)
         self.assertEqual(recurring.account.currency.code, "USD")

app/apps/transactions/tests/test_views.py

@@ -1,174 +0,0 @@
-from datetime import date
-
-from django.contrib.auth import get_user_model
-from django.test import TestCase, override_settings
-from django.utils import timezone
-
-from apps.accounts.models import Account, AccountGroup
-from apps.currencies.models import Currency
-from apps.transactions.models import (
-    Transaction,
-    TransactionCategory,
-    TransactionTag,
-)
-
-
-@override_settings(
-    STORAGES={
-        "default": {"BACKEND": "django.core.files.storage.FileSystemStorage"},
-        "staticfiles": {"BACKEND": "django.contrib.staticfiles.storage.StaticFilesStorage"},
-    },
-    WHITENOISE_AUTOREFRESH=True,
-)
-class TransactionSimpleAddViewTests(TestCase):
-    """Tests for the transaction_simple_add view with query parameters"""
-
-    def setUp(self):
-        """Set up test data"""
-        User = get_user_model()
-        self.user = User.objects.create_user(
-            email="testuser@test.com", password="testpass123"
-        )
-        self.client.login(username="testuser@test.com", password="testpass123")
-
-        self.currency = Currency.objects.create(
-            code="USD", name="US Dollar", decimal_places=2, prefix="$ "
-        )
-        self.account_group = AccountGroup.objects.create(name="Test Group")
-        self.account = Account.objects.create(
-            name="Test Account", group=self.account_group, currency=self.currency
-        )
-        self.category = TransactionCategory.objects.create(name="Test Category")
-        self.tag = TransactionTag.objects.create(name="TestTag")
-
-    def test_get_returns_form_with_default_values(self):
-        """Test GET request returns 200 and form with defaults"""
-        response = self.client.get("/add/")
-        self.assertEqual(response.status_code, 200)
-        self.assertIn("form", response.context)
-
-    def test_get_with_type_param(self):
-        """Test type param sets form initial value"""
-        response = self.client.get("/add/?type=EX")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("type"), Transaction.Type.EXPENSE)
-
-    def test_get_with_account_param(self):
-        """Test account param sets form initial value"""
-        response = self.client.get(f"/add/?account={self.account.id}")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("account"), self.account.id)
-
-    def test_get_with_is_paid_param_true(self):
-        """Test is_paid param with true value"""
-        response = self.client.get("/add/?is_paid=true")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertTrue(form.initial.get("is_paid"))
-
-    def test_get_with_is_paid_param_false(self):
-        """Test is_paid param with false value"""
-        response = self.client.get("/add/?is_paid=false")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertFalse(form.initial.get("is_paid"))
-
-    def test_get_with_amount_param(self):
-        """Test amount param sets form initial value"""
-        response = self.client.get("/add/?amount=150.50")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("amount"), "150.50")
-
-    def test_get_with_description_param(self):
-        """Test description param sets form initial value"""
-        response = self.client.get("/add/?description=Test%20Transaction")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("description"), "Test Transaction")
-
-    def test_get_with_notes_param(self):
-        """Test notes param sets form initial value"""
-        response = self.client.get("/add/?notes=Some%20notes")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("notes"), "Some notes")
-
-    def test_get_with_category_param(self):
-        """Test category param sets form initial value"""
-        response = self.client.get(f"/add/?category={self.category.id}")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("category"), self.category.id)
-
-    def test_get_with_tags_param(self):
-        """Test tags param as comma-separated names"""
-        response = self.client.get("/add/?tags=TestTag,AnotherTag")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("tags"), ["TestTag", "AnotherTag"])
-
-    def test_get_with_all_params(self):
-        """Test all params together work correctly"""
-        url = (
-            f"/add/?type=EX&account={self.account.id}&is_paid=true"
-            f"&amount=200.00&description=Full%20Test&notes=Test%20notes"
-            f"&category={self.category.id}&tags=TestTag"
-        )
-        response = self.client.get(url)
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("type"), Transaction.Type.EXPENSE)
-        self.assertEqual(form.initial.get("account"), self.account.id)
-        self.assertTrue(form.initial.get("is_paid"))
-        self.assertEqual(form.initial.get("amount"), "200.00")
-        self.assertEqual(form.initial.get("description"), "Full Test")
-        self.assertEqual(form.initial.get("notes"), "Test notes")
-        self.assertEqual(form.initial.get("category"), self.category.id)
-        self.assertEqual(form.initial.get("tags"), ["TestTag"])
-
-    def test_post_creates_transaction(self):
-        """Test form submission creates transaction"""
-        data = {
-            "account": self.account.id,
-            "type": "EX",
-            "is_paid": True,
-            "date": timezone.now().date().isoformat(),
-            "amount": "100.00",
-            "description": "Test Transaction",
-        }
-        response = self.client.post("/add/", data)
-        self.assertEqual(response.status_code, 200)
-        self.assertTrue(
-            Transaction.objects.filter(description="Test Transaction").exists()
-        )
-
-    def test_get_with_date_param(self):
-        """Test date param overrides expected date"""
-        response = self.client.get("/add/?date=2025-06-15")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("date"), date(2025, 6, 15))
-
-    def test_get_with_reference_date_param(self):
-        """Test reference_date param sets form initial value"""
-        response = self.client.get("/add/?reference_date=2025-07-01")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("reference_date"), date(2025, 7, 1))
-
-    def test_get_with_account_name_param(self):
-        """Test account param by name (case-insensitive)"""
-        response = self.client.get("/add/?account=Test%20Account")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("account"), self.account.id)
-
-    def test_get_with_category_name_param(self):
-        """Test category param by name (case-insensitive)"""
-        response = self.client.get("/add/?category=Test%20Category")
-        self.assertEqual(response.status_code, 200)
-        form = response.context["form"]
-        self.assertEqual(form.initial.get("category"), self.category.id)

app/apps/transactions/views/quick_transactions.py

@@ -137,7 +137,6 @@ def quick_transaction_add_as_transaction(request, quick_transaction_id):
             "category",
             "tags",
             "entities",
-            "internal_id",
         ],
     )
 
@@ -207,7 +206,6 @@ def quick_transaction_add_as_quick_transaction(request, transaction_id):
             "recurring_transaction",
             "deleted",
             "deleted_at",
-            "internal_id",
         ],
     )
 

app/apps/transactions/views/transactions.py

@@ -142,95 +142,26 @@ def transaction_simple_add(request):
         year=year,
     ).date()
 
-    # Build initial data from query parameters
-    initial_data = {
-        "date": expected_date,
-        "type": transaction_type,
-    }
-
-    # Handle date param (ISO format: YYYY-MM-DD) - overrides expected_date
-    date_param = request.GET.get("date")
-    if date_param:
-        try:
-            initial_data["date"] = datetime.datetime.strptime(date_param, "%Y-%m-%d").date()
-        except ValueError:
-            pass
-
-    # Handle reference_date param (ISO format: YYYY-MM-DD)
-    reference_date_param = request.GET.get("reference_date")
-    if reference_date_param:
-        try:
-            initial_data["reference_date"] = datetime.datetime.strptime(reference_date_param, "%Y-%m-%d").date()
-        except ValueError:
-            pass
-
-    # Handle account param (by ID or name)
-    account_param = request.GET.get("account")
-    if account_param:
-        try:
-            initial_data["account"] = int(account_param)
-        except (ValueError, TypeError):
-            # Try to find by name
-            from apps.accounts.models import Account
-            account = Account.objects.filter(name__iexact=account_param, is_archived=False).first()
-            if account:
-                initial_data["account"] = account.pk
-
-    # Handle is_paid param (boolean)
-    is_paid = request.GET.get("is_paid")
-    if is_paid is not None:
-        initial_data["is_paid"] = is_paid.lower() in ("true", "1", "yes")
-
-    # Handle amount param (decimal)
-    amount = request.GET.get("amount")
-    if amount:
-        try:
-            initial_data["amount"] = amount
-        except (ValueError, TypeError):
-            pass
-
-    # Handle description param (string)
-    description = request.GET.get("description")
-    if description:
-        initial_data["description"] = description
-
-    # Handle notes param (string)
-    notes = request.GET.get("notes")
-    if notes:
-        initial_data["notes"] = notes
-
-    # Handle category param (by ID or name)
-    category_param = request.GET.get("category")
-    if category_param:
-        try:
-            initial_data["category"] = int(category_param)
-        except (ValueError, TypeError):
-            # Try to find by name
-            from apps.transactions.models import TransactionCategory
-            category = TransactionCategory.objects.filter(name__iexact=category_param, active=True).first()
-            if category:
-                initial_data["category"] = category.pk
-
-    # Handle tags param (comma-separated names)
-    tags = request.GET.get("tags")
-    if tags:
-        initial_data["tags"] = [t.strip() for t in tags.split(",") if t.strip()]
-
-    # Handle entities param (comma-separated names)
-    entities = request.GET.get("entities")
-    if entities:
-        initial_data["entities"] = [e.strip() for e in entities.split(",") if e.strip()]
-
     if request.method == "POST":
         form = TransactionForm(request.POST)
         if form.is_valid():
             form.save()
             messages.success(request, _("Transaction added successfully"))
-            # Only reset form after successful save
-            form = TransactionForm(initial=initial_data)
+
+        form = TransactionForm(
+            initial={
+                "date": expected_date,
+                "type": transaction_type,
+            },
+        )
 
     else:
-        form = TransactionForm(initial=initial_data)
+        form = TransactionForm(
+            initial={
+                "date": expected_date,
+                "type": transaction_type,
+            },
+        )
 
     return render(
         request,
@@ -282,7 +213,6 @@ def transactions_bulk_edit(request):
         if form.is_valid():
             # Apply changes from the form to all selected transactions
             for transaction in transactions:
-                old_data = deepcopy(transaction)
                 for field_name, value in form.cleaned_data.items():
                     if value or isinstance(
                         value, bool
@@ -295,7 +225,7 @@ def transactions_bulk_edit(request):
                             setattr(transaction, field_name, value)
 
                 transaction.save()
-                transaction_updated.send(sender=transaction, old_data=old_data)
+                transaction_updated.send(sender=transaction)
 
             messages.success(
                 request,
@@ -443,13 +373,10 @@ def transactions_transfer(request):
 @require_http_methods(["GET"])
 def transaction_pay(request, transaction_id):
     transaction = get_object_or_404(Transaction, pk=transaction_id)
-    old_data = deepcopy(transaction)
-
     new_is_paid = False if transaction.is_paid else True
     transaction.is_paid = new_is_paid
     transaction.save()
-
-    transaction_updated.send(sender=transaction, old_data=old_data)
+    transaction_updated.send(sender=transaction)
 
     response = render(
         request,
@@ -467,12 +394,11 @@ def transaction_pay(request, transaction_id):
 @require_http_methods(["GET"])
 def transaction_mute(request, transaction_id):
     transaction = get_object_or_404(Transaction, pk=transaction_id)
-    old_data = deepcopy(transaction)
 
     new_mute = False if transaction.mute else True
     transaction.mute = new_mute
     transaction.save()
-    transaction_updated.send(sender=transaction, old_data=old_data)
+    transaction_updated.send(sender=transaction)
 
     response = render(
         request,
@@ -488,20 +414,19 @@ def transaction_mute(request, transaction_id):
 @require_http_methods(["GET"])
 def transaction_change_month(request, transaction_id, change_type):
     transaction: Transaction = get_object_or_404(Transaction, pk=transaction_id)
-    old_data = deepcopy(transaction)
 
     if change_type == "next":
         transaction.reference_date = transaction.reference_date + relativedelta(
             months=1
         )
         transaction.save()
-        transaction_updated.send(sender=transaction, old_data=old_data)
+        transaction_updated.send(sender=transaction)
     elif change_type == "previous":
         transaction.reference_date = transaction.reference_date - relativedelta(
             months=1
         )
         transaction.save()
-        transaction_updated.send(sender=transaction, old_data=old_data)
+        transaction_updated.send(sender=transaction)
 
     return HttpResponse(
         status=204,
@@ -515,11 +440,9 @@ def transaction_change_month(request, transaction_id, change_type):
 def transaction_move_to_today(request, transaction_id):
     transaction: Transaction = get_object_or_404(Transaction, pk=transaction_id)
 
-    old_data = deepcopy(transaction)
-
     transaction.date = timezone.localdate(timezone.now())
     transaction.save()
-    transaction_updated.send(sender=transaction, old_data=old_data)
+    transaction_updated.send(sender=transaction)
 
     return HttpResponse(
         status=204,
@@ -666,10 +589,7 @@ def transaction_all_currency_summary(request):
 
     f = TransactionsFilter(request.GET, queryset=transactions)
 
-    currency_data = calculate_currency_totals(
-        f.qs.exclude(account__in=request.user.untracked_accounts.all()),
-        ignore_empty=True,
-    )
+    currency_data = calculate_currency_totals(f.qs.all(), ignore_empty=True)
     currency_percentages = calculate_percentage_distribution(currency_data)
 
     context = {

app/apps/users/forms.py

@@ -1,43 +1,35 @@
-from apps.common.middleware.thread_local import get_current_user
-from apps.common.widgets.crispy.submit import NoClassSubmit
-from apps.users.models import UserSettings
 from crispy_forms.bootstrap import (
     FormActions,
 )
 from crispy_forms.helper import FormHelper
-from crispy_forms.layout import HTML, Column, Div, Field, Layout, Row, Submit
+from crispy_forms.layout import Layout, Submit, Row, Column, Field, Div, HTML
 from django import forms
 from django.contrib.auth import get_user_model
 from django.contrib.auth.forms import (
+    UsernameField,
     AuthenticationForm,
     UserCreationForm,
-    UsernameField,
 )
 from django.db import transaction
 from django.utils.translation import gettext_lazy as _
 
+from apps.common.widgets.crispy.submit import NoClassSubmit
+from apps.users.models import UserSettings
+from apps.common.middleware.thread_local import get_current_user
+
 
 class LoginForm(AuthenticationForm):
     username = UsernameField(
         label=_("E-mail"),
         widget=forms.EmailInput(
-            attrs={
-                "class": "input",
-                "placeholder": _("E-mail"),
-                "name": "email",
-                "autocomplete": "email",
-            }
+            attrs={"class": "form-control", "placeholder": "E-mail", "name": "email"}
         ),
     )
     password = forms.CharField(
         label=_("Password"),
         strip=False,
         widget=forms.PasswordInput(
-            attrs={
-                "class": "input",
-                "placeholder": _("Password"),
-                "autocomplete": "current-password",
-            }
+            attrs={"class": "form-control", "placeholder": "Senha"}
         ),
     )
 
@@ -53,7 +45,7 @@ class LoginForm(AuthenticationForm):
         self.helper.layout = Layout(
             "username",
             "password",
-            Submit("Submit", "Login", css_class="w-full mt-3"),
+            Submit("Submit", "Login", css_class="btn btn-primary w-100"),
         )
 
 
@@ -97,8 +89,6 @@ class UserSettingsForm(forms.ModelForm):
         ("AA", _("Default")),
         ("DC", "1.234,50"),
         ("CD", "1,234.50"),
-        ("SD", "1 234.50"),
-        ("SC", "1 234,50"),
     ]
 
     date_format = forms.ChoiceField(
@@ -137,16 +127,18 @@ class UserSettingsForm(forms.ModelForm):
         self.helper.layout = Layout(
             "language",
             "timezone",
-            HTML('<hr class="hr my-3" />'),
+            HTML("<hr />"),
             "date_format",
             "datetime_format",
             "number_format",
-            HTML('<hr class="hr my-3" />'),
+            HTML("<hr />"),
             "start_page",
-            HTML('<hr class="hr my-3" />'),
+            HTML("<hr />"),
             "volume",
             FormActions(
-                NoClassSubmit("submit", _("Save"), css_class="btn btn-primary"),
+                NoClassSubmit(
+                    "submit", _("Save"), css_class="btn btn-outline-primary w-100"
+                ),
             ),
         )
 
@@ -197,8 +189,8 @@ class UserUpdateForm(forms.ModelForm):
         # Define the layout using Crispy Forms, including the new fields
         self.helper.layout = Layout(
             Row(
-                Column("first_name"),
-                Column("last_name"),
+                Column("first_name", css_class="form-group col-md-6"),
+                Column("last_name", css_class="form-group col-md-6"),
                 css_class="row",
             ),
             Field("email"),
@@ -219,13 +211,17 @@ class UserUpdateForm(forms.ModelForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 
@@ -356,8 +352,8 @@ class UserAddForm(UserCreationForm):
         self.helper.layout = Layout(
             Field("email"),
             Row(
-                Column("first_name"),
-                Column("last_name"),
+                Column("first_name", css_class="form-group col-md-6"),
+                Column("last_name", css_class="form-group col-md-6"),
                 css_class="row",
             ),
             # UserCreationForm provides 'password1' and 'password2' fields
@@ -377,13 +373,17 @@ class UserAddForm(UserCreationForm):
         if self.instance and self.instance.pk:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Update"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Update"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
         else:
             self.helper.layout.append(
                 FormActions(
-                    NoClassSubmit("submit", _("Add"), css_class="btn btn-primary"),
+                    NoClassSubmit(
+                        "submit", _("Add"), css_class="btn btn-outline-primary w-100"
+                    ),
                 ),
             )
 

app/apps/users/urls.py

@@ -4,6 +4,7 @@ from . import views
 
 urlpatterns = [
     path("", views.index, name="index"),
+    path("setup/", views.setup, name="setup"),
     path("login/", views.UserLoginView.as_view(), name="login"),
     # path("login/fallback/", views.UserLoginView.as_view(), name="fallback_login"),
     path("logout/", views.logout_view, name="logout"),
@@ -17,16 +18,6 @@ urlpatterns = [
         views.toggle_sound_playing,
         name="toggle_sound_playing",
     ),
-    path(
-        "user/session/toggle-sidebar/",
-        views.toggle_sidebar_status,
-        name="toggle_sidebar_status",
-    ),
-    path(
-        "user/session/toggle-theme/",
-        views.toggle_theme,
-        name="toggle_theme",
-    ),
     path(
         "user/settings/",
         views.update_settings,

app/apps/users/views.py

@@ -1,26 +1,29 @@
-from apps.common.decorators.demo import disabled_on_demo
-from apps.common.decorators.htmx import only_htmx
-from apps.common.decorators.user import htmx_login_required, is_superuser
-from apps.users.forms import (
-    LoginForm,
-    UserAddForm,
-    UserSettingsForm,
-    UserUpdateForm,
-)
-from apps.users.models import UserSettings
 from django.contrib import messages
-from django.contrib.auth import get_user_model, logout
+from django.contrib.auth import logout, get_user_model
 from django.contrib.auth.decorators import login_required
 from django.contrib.auth.views import (
     LoginView,
 )
 from django.core.exceptions import PermissionDenied
 from django.http import HttpResponse
-from django.shortcuts import get_object_or_404, redirect, render
+from django.shortcuts import redirect, render, get_object_or_404
 from django.urls import reverse
 from django.utils.translation import gettext_lazy as _
 from django.views.decorators.http import require_http_methods
 
+from apps.common.decorators.htmx import only_htmx
+from apps.common.decorators.user import is_superuser, htmx_login_required
+from apps.users.forms import (
+    LoginForm,
+    UserSettingsForm,
+    UserUpdateForm,
+    UserAddForm,
+)
+from apps.users.models import UserSettings
+from apps.common.decorators.demo import disabled_on_demo
+from apps.currencies.models import Currency
+from apps.accounts.models import Account
+
 
 def logout_view(request):
     logout(request)
@@ -47,6 +50,28 @@ def index(request):
         return redirect(reverse("monthly_index"))
 
 
+@login_required
+def setup(request):
+    has_currency = Currency.objects.exists()
+    has_account = Account.objects.exists()
+    # return render(
+    #     request,
+    #     "users/setup/setup.html",
+    #     {"has_currency": has_currency, "has_account": has_account},
+    # )
+    if not has_currency or not has_account:
+        return render(
+            request,
+            "users/setup/setup.html",
+            {"has_currency": has_currency, "has_account": has_account},
+        )
+    else:
+        return HttpResponse(
+            status=200,
+            headers={"HX-Reswap": "delete"},
+        )
+
+
 class UserLoginView(LoginView):
     form_class = LoginForm
     template_name = "users/login.html"
@@ -115,43 +140,6 @@ def update_settings(request):
     return render(request, "users/fragments/user_settings.html", {"form": form})
 
 
-@only_htmx
-@htmx_login_required
-@require_http_methods(["GET"])
-def toggle_sidebar_status(request):
-    if not request.session.get("sidebar_status"):
-        request.session["sidebar_status"] = "floating"
-
-    if request.session["sidebar_status"] == "floating":
-        request.session["sidebar_status"] = "fixed"
-    elif request.session["sidebar_status"] == "fixed":
-        request.session["sidebar_status"] = "floating"
-    else:
-        request.session["sidebar_status"] = "fixed"
-
-    return HttpResponse(
-        status=204,
-    )
-
-
-@htmx_login_required
-@require_http_methods(["GET"])
-def toggle_theme(request):
-    if not request.session.get("theme"):
-        request.session["theme"] = "wygiwyh_dark"
-
-    if request.session["theme"] == "wygiwyh_dark":
-        request.session["theme"] = "wygiwyh_light"
-    elif request.session["theme"] == "wygiwyh_light":
-        request.session["theme"] = "wygiwyh_dark"
-    else:
-        request.session["theme"] = "wygiwyh_light"
-
-    return HttpResponse(
-        status=204,
-    )
-
-
 @htmx_login_required
 @is_superuser
 @require_http_methods(["GET"])

app/apps/yearly_overview/views.py

@@ -79,7 +79,7 @@ def yearly_overview_by_currency(request, year: int):
     currency = request.GET.get("currency")
 
     # Base query filter
-    filter_params = {"reference_date__year": year}
+    filter_params = {"reference_date__year": year, "account__is_archived": False}
 
     # Add month filter if provided
     if month:
@@ -95,7 +95,6 @@ def yearly_overview_by_currency(request, year: int):
     transactions = (
         Transaction.objects.filter(**filter_params)
         .exclude(Q(Q(category__mute=True) & ~Q(category=None)) | Q(mute=True))
-        .exclude(account__in=request.user.untracked_accounts.all())
         .order_by("account__currency__name")
     )