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

feat: users csv import #1409

Open
wants to merge 18 commits into
base: development
Choose a base branch
from

Conversation

jorwoods
Copy link
Contributor

@jorwoods jorwoods commented Jun 28, 2024

This adds functionality to bulk add and bulk remove users. It has the TSC users pass in an Iterable[UserItem], and creates the csv file formatted as specified by CSV guidelines.

The previous version of create_users_from_file read a CSV file, and called the add endpoint for each one. This PR deprecates that endpoint in favor of directing users to bulk_add.

@jorwoods jorwoods marked this pull request as draft June 28, 2024 16:04
@jorwoods jorwoods marked this pull request as ready for review June 29, 2024 03:08
@jorwoods jorwoods changed the title Jorwoods/users csv import feat: users csv import Jul 16, 2024
@jorwoods jorwoods force-pushed the jorwoods/users_csv_import branch 2 times, most recently from 1e1e33f to 0152fae Compare July 26, 2024 12:26
@jorwoods jorwoods force-pushed the jorwoods/users_csv_import branch 3 times, most recently from 257485a to cff39c1 Compare September 19, 2024 09:17
@jacalata
Copy link
Contributor

I've been slow on this one because I have a lot of this code in tabcmd and have been trying to validate against that. I'll make some progress on it soon!

@jorwoods
Copy link
Contributor Author

jorwoods commented Nov 7, 2024

I've been slow on this one because I have a lot of this code in tabcmd and have been trying to validate against that. I'll make some progress on it soon!

@jacalata Any thoughts on this?

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

Successfully merging this pull request may close these issues.

2 participants