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

Support additional export parameters when exporting from Admin UI action #1671

Conversation

matthewhegarty
Copy link
Contributor

@matthewhegarty matthewhegarty commented Oct 31, 2023

Problem

Closes #1440

The issue is that full export workflow (selecting resource and file format) is not presented when exporting via actions. This adds the export workflow (which can be disabled if required).

The change is summarised here

Solution

If exporting from an action, selected item primary keys are read. The export action then renders the standard 'export' page, with chosen resources as a hidden field.

This removes the need for the 'file format' and 'resource' drop downs in the action UI (it doesn't really belong here anyway).

Additional changes

  • Tidied up documentation
  • Removed action_formats.js - no longer required as a result of this change.

Acceptance Criteria

  • Manual testing
  • Automated testing
  • Documentation updates

@matthewhegarty matthewhegarty marked this pull request as ready for review November 14, 2023 16:20
Copy link

@pokken-magic pokken-magic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a very good idea to me! Nice work.

@matthewhegarty matthewhegarty merged commit b210310 into django-import-export:release-4 Nov 14, 2023
4 of 5 checks passed
@matthewhegarty matthewhegarty deleted the issue-1440-export-multiple-resources branch November 14, 2023 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants