feat: add support for de-provisioning users #960
Conversation
edward-ly
force-pushed
the
feat/de-provision-user
branch
from
45936f2 to
e43c364
Compare
Signed-off-by: Edward Ly <[email protected]>
edward-ly
force-pushed
the
feat/de-provision-user
branch
from
e43c364 to
7013ec3
Compare
There was a problem hiding this comment.
@NoCSRFRequired is missing.
We could also make this an OCS controller. If you want to do it in this PR you can, otherwise we'll do it later.
Also, if a 404 is returned, I think it's better if the payload is empty or something like ['error' => 'User not found'].
Signed-off-by: Edward Ly <[email protected]>
|
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! |
Resolves #902.