Skip to content

Conversation

bmihelac
Copy link
Member

@bmihelac bmihelac commented Nov 5, 2024

Problem

Feature proposal - import and export management command

Solution

As described. Additional encoding argument is added to both commands.

Acceptance Criteria

Have you written tests? Have you included screenshots of your changes if applicable?

yes

Did you document your changes?

yes

@bmihelac bmihelac force-pushed the feat/management-commands branch from e2e4167 to 224a54a Compare November 5, 2024 10:59
@coveralls
Copy link

coveralls commented Nov 5, 2024

Coverage Status

coverage: 100.0%. remained the same
when pulling a1bbb94 on bmihelac:feat/management-commands
into 5e3cdd8 on django-import-export:main.

@bmihelac bmihelac force-pushed the feat/management-commands branch from 477a7c9 to 643d08a Compare November 5, 2024 12:22
Copy link
Contributor

@matthewhegarty matthewhegarty left a comment

Choose a reason for hiding this comment

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

This looks great! Thanks for your hard work. I tested and added a few review comments.

@bmihelac bmihelac force-pushed the feat/management-commands branch from 6360ca8 to f9feca5 Compare November 5, 2024 12:31
@bmihelac bmihelac force-pushed the feat/management-commands branch from bed3e66 to a5bf523 Compare November 5, 2024 12:40
@matthewhegarty matthewhegarty linked an issue Nov 5, 2024 that may be closed by this pull request
@bmihelac bmihelac force-pushed the feat/management-commands branch from 3eb645b to 2a17796 Compare November 5, 2024 15:15
@bmihelac bmihelac force-pushed the feat/management-commands branch from cab9129 to 99a5057 Compare November 5, 2024 15:43
Copy link
Contributor

@matthewhegarty matthewhegarty left a comment

Choose a reason for hiding this comment

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

Added some minor points about language in documentation.

@bmihelac bmihelac force-pushed the feat/management-commands branch from 438b17b to 0989f83 Compare November 6, 2024 06:43
@andrewgy8
Copy link
Member

Good work @bmihelac ! Lets see how the community uses this! 😄

@bmihelac
Copy link
Member Author

bmihelac commented Nov 7, 2024

It would be a good idea to add a confirmation prompt for the import command that would be skipped if the --noinput option is passed.

@matthewhegarty matthewhegarty self-requested a review November 9, 2024 12:47
Copy link
Contributor

@matthewhegarty matthewhegarty left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@bmihelac bmihelac merged commit c4d1c4b into django-import-export:main Nov 19, 2024
14 checks passed
@bmihelac
Copy link
Member Author

Merged - @matthewhegarty and @andrewgy8 thanks for review!

Viicos added a commit to Viicos/typeshed that referenced this pull request Jan 19, 2025
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.

Feature proposal - import and export management commands

4 participants