Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add BackgroundJobExportMixin #133

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

sky-code
Copy link

@sky-code sky-code commented Sep 30, 2024

Fully functional concept of possible fix for #111. It saves the queryset from export page and use it when export performed later in a job.

I don't know better way to save/restore queryset between initial request and celery job, too many points in django where queryset can be dynamically modified.

In case of python or django upgrade, if serialized with pickle query breaks it can be re-created from saved queryString.

Select/unselect fields to be exported are not supported, all fields will be exported in current implementation but this feature can be added.

django-import-export and django-import-export-celery select resources differently, some work needs to be done for unification and better integration of feature.

Trigger of run_export_job.delay needs to be updated and separated from creation and scheduling task execution to be able to create job without scheduling it automatically, to set non default values for Site of origin: and Send me an email when this export job is complete and than schedule job after that.

See also #130

@aroncds
Copy link

aroncds commented Oct 9, 2024

it is missing forms.py

@sky-code
Copy link
Author

sky-code commented Oct 9, 2024

it is missing forms.py

not sure what you mean

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants