-
Notifications
You must be signed in to change notification settings - Fork 1
Add bulk user import functionality from CSV #63
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds bulk user import functionality to create multiple IMAP synchronization tasks from a CSV file. The implementation includes a Python script that reads user credentials from CSV, validates the input, and calls the existing API to create tasks programmatically.
Key changes:
- New
create_tasks_from_csv.pyscript with CSV parsing, validation, and batch task creation capabilities - Sample
users.csvfile with 5 example user records demonstrating the expected format - Documentation in README.md covering CSV format, usage examples, and feature descriptions
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 14 comments.
| File | Description |
|---|---|
| users.csv | Sample CSV file with 5 user records showing the required column structure and format |
| create_tasks_from_csv.py | Main Python script implementing CSV parsing, validation, delimiter detection, and API integration for bulk task creation |
| README.md | Documentation section covering bulk import usage, CSV format requirements, command-line options, and example output |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 13 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 10 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
DavidePrincipi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, it's a nice feature: let's document it in the Admin's manual too.
Introduce a new script for creating synchronization tasks from a CSV file containing user credentials. Include an initial users.csv file with sample data and update the README to document the new bulk import feature and its usage.
asked by @nrauso
NethServer/dev#7791