Merge pull request #151 from saritasa-nest/feature/allow-skip-export-confirmation-page

Add support to skip export confirmation page with settings

Author: Eg0ra

HISTORY.rst

@@ -9,6 +9,10 @@ unreleased
 * Add ability to customize the order of operation for export queryset
 * Add ability to set dataset title
 * Add field selection to the export confirmation page in the admin panel
+* Add support to skip export confirmation page in admin (https://github.com/saritasa-nest/django-import-export-extensions/issues/122):
+
+  * with `IMPORT_EXPORT_SKIP_ADMIN_EXPORT_UI=True` setting from original package
+  * with `skip_export_form` flag at the admin model level
 
 1.8.0 (2025-06-30)
 ------------------

import_export_extensions/admin/mixins/export_mixin.py

@@ -71,6 +71,7 @@ class CeleryExportAdminMixin(
     )
     get_export_form_class = import_export_admin.ExportMixin.get_export_form_class  # noqa
     get_export_resource_fields_from_form = import_export_admin.ExportMixin.get_export_resource_fields_from_form  # noqa
+    is_skip_export_form_enabled = import_export_admin.ExportMixin.is_skip_export_form_enabled  # noqa
 
     def get_export_context_data(self, **kwargs):
         """Get context data for export."""
@@ -126,17 +127,30 @@ def celery_export_action(self, request, *args, **kwargs):
             raise PermissionDenied
 
         formats = self.get_export_formats()
+        resource_kwargs = self.get_export_resource_kwargs(
+            *args,
+            **kwargs,
+            request=request,
+        )
+        if self.is_skip_export_form_enabled():
+            job = self.create_export_job(
+                request=request,
+                resource_class=self.get_export_resource_classes(request)[0],
+                resource_kwargs=resource_kwargs,
+                file_format=formats[0],
+            )
+            return self._redirect_to_export_status_page(
+                request=request,
+                job=job,
+            )
+
         form_type = self.get_export_form_class()
         form = form_type(
             formats=formats,
             resources=self.get_export_resource_classes(request),
             data=request.POST or None,
         )
-        resource_kwargs = self.get_export_resource_kwargs(
-            *args,
-            **kwargs,
-            request=request,
-        )
+
         if request.method == "POST" and form.is_valid():
             file_format = formats[int(form.cleaned_data["format"])]
             # Get the selected export fields from the form

test_project/tests/integration_tests/test_admin/test_export/test_export.py

@@ -2,6 +2,7 @@
 import pathlib
 import uuid
 
+import django.test
 from django.contrib.auth.models import User
 from django.test.client import Client
 from django.urls import reverse
@@ -326,3 +327,31 @@ def test_export_using_get_params(
         instrument := Instrument.objects.filter(id=int(record[-1])).first()
     )
     assert instrument.title == instrument_title
+
+
+@django.test.override_settings(
+    IMPORT_EXPORT_SKIP_ADMIN_EXPORT_UI=True,
+)
+@pytest.mark.usefixtures("existing_artist")
+@pytest.mark.django_db(transaction=True)
+def test_export_skip_confirmation_page(client: Client, superuser: User):
+    """Test export using Django Admin, skipping confirmation page."""
+    client.force_login(superuser)
+
+    # Make get request to admin export page
+    response = client.get(
+        path=reverse("admin:fake_app_artist_export"),
+    )
+    assert response.status_code == status.HTTP_302_FOUND
+
+    status_response = client.get(response.url)
+    assert status_response.status_code == status.HTTP_302_FOUND
+
+    result_response = client.get(status_response.url)
+    assert result_response.status_code == status.HTTP_200_OK
+
+    assert ExportJob.objects.exists()
+    export_job = ExportJob.objects.first()
+    assert export_job.export_status == ExportJob.ExportStatus.EXPORTED, (
+        export_job.traceback
+    )