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

Also remove /var/mail/<username> directory on user delete (with --purge option) #602

Merged
merged 2 commits into from Dec 20, 2018

Conversation

Projects
None yet
3 participants
@eauchat
Copy link

eauchat commented Dec 16, 2018

The problem

A user's emails weren't removed on user deletion.

PR Status

...

How to test

Delete a user, either from CLI or from API.

Validation

  • Principle agreement 0/2 :
  • Quick review 0/1 :
  • Simple test 0/1 :
  • Deep review 0/1 :
@zamentur
Copy link
Contributor

zamentur left a comment

LGTM

@zamentur

This comment has been minimized.

Copy link
Contributor

zamentur commented Dec 20, 2018

Thanks for this fix :)

@alexAubin
Copy link
Member

alexAubin left a comment

LGTM as well, thanks 👍

@alexAubin alexAubin merged commit 3e1064a into YunoHost:stretch-unstable Dec 20, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment