-
-
Notifications
You must be signed in to change notification settings - Fork 273
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
Quickfixs #148
Quickfixs #148
Conversation
I still need to make some tests, don't merge too quickly :$ |
@@ -200,12 +194,13 @@ def domain_remove(auth, domain, force=False): | |||
m18n.n('domain_uninstall_app_first')) | |||
|
|||
if auth.remove('virtualdomain=' + domain + ',ou=domains') or force: | |||
os.system('rm -rf /etc/yunohost/certs/%s' % domain) | |||
shutil.rmtree('rm -rf /etc/yunohost/certs/%s' % domain) |
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.
Oops, it seems there is an issue here! :)
Thank you for this really big review!! :) We will have to see in which branch it should be merged... |
What is the status of this PR ? |
Probably too big and outdated now, I'm closing it and I'll redo one or a series of smaller one (or directly push on unstable if it's only things like pep8 which are just looks and not possible breaking things at a time where we don't have too much PRs). |
Hello,
A serie of quickfix I've made while reading this file (I wanted to know how domains were stored).
You can cherry-pick if you want.