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 django-import-export for organization users #4986

Merged
merged 2 commits into from
Sep 23, 2024
Merged

Conversation

bufke
Copy link
Contributor

@bufke bufke commented Jun 26, 2024

Description

Django admin can now import and export organization users to and from formats including csv.

Notes

It's a common need to import and export users. Django-import-export provides a customizable way to do so with minimal code. It's IMO a well maintained package but does occasionally include minor breaking changes, such as the 4.x release.

This change took about 45 minutes and was tested with importing Organization Users.

Copy link
Member

@jnm jnm left a comment

Choose a reason for hiding this comment

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

The changes look fine, but the PR description does not reflect well what the code actually does. See my comment on https://www.notion.so/kobotoolbox/Import-billing-users-in-bulk-c389e89af9b94606af6fef069facf850.

I have not run the branch and tested the new functionality.

@bufke bufke requested a review from jnm September 20, 2024 20:08
Copy link
Member

@jnm jnm left a comment

Choose a reason for hiding this comment

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

Thanks

@jnm jnm changed the title Add django import export Add django-import-export for organization users Sep 23, 2024
@jnm jnm merged commit 173410a into beta Sep 23, 2024
5 checks passed
@jnm jnm deleted the django-import-export branch September 23, 2024 17:29
jnm added a commit that referenced this pull request Oct 3, 2024
Add subdependencies that were missing from #4986, presumably as a result
of manually editing `*requirements.txt` files instead of running `pip-compile.sh`
@jnm jnm mentioned this pull request Oct 3, 2024
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants