Skip to content

Conversation

@come-nc
Copy link
Collaborator

@come-nc come-nc commented Dec 2, 2025

Exporting a user from occ now stores the date of last export in the database. A new command user_migration:manage allows to list exported users and delete them as a batch.

@come-nc come-nc self-assigned this Dec 2, 2025
@come-nc come-nc added 3. to review Waiting for reviews enhancement New feature request labels Dec 2, 2025
Exporting a user from occ now stores the date of last export in the
 database. A new command user_migration:manage allows to list exported
 users and delete them as a batch.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc come-nc force-pushed the feat/add-export-date-and-manage-command branch from 3f88e59 to 7f428b1 Compare December 2, 2025 10:28
@come-nc come-nc marked this pull request as ready for review December 2, 2025 10:28
Copy link
Member

@provokateurin provokateurin left a comment

Choose a reason for hiding this comment

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

This feels really dangerous. What if you exported some users earlier and then want to delete newly exported users, but there is no way to prevent the earlier users being deleted as well?

Co-authored-by: Kate <26026535+provokateurin@users.noreply.github.com>
Signed-off-by: Côme Chilliet <91878298+come-nc@users.noreply.github.com>
@come-nc
Copy link
Collaborator Author

come-nc commented Dec 2, 2025

This feels really dangerous. What if you exported some users earlier and then want to delete newly exported users, but there is no way to prevent the earlier users being deleted as well?

Do you have a proposition to fix that ?
It’s always allowed to not use the --delete option and rely on user:delete to delete each user by hand.

Would it feel less dangerous if I add an option to filter by minimum date? --since="1 week ago"?

@provokateurin
Copy link
Member

Would it feel less dangerous if I add an option to filter by minimum date? --since="1 week ago"?

Either this, or a flag to filter by exact export date (where the export command would set all accounts to the same value an print it at the end).

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
@come-nc
Copy link
Collaborator Author

come-nc commented Dec 15, 2025

Added a --since option 👍

@provokateurin provokateurin merged commit 3f404eb into main Dec 15, 2025
46 checks passed
@provokateurin provokateurin deleted the feat/add-export-date-and-manage-command branch December 15, 2025 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement New feature request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants