-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Update site.py #2306
Update site.py #2306
Conversation
Fixed --db-name ignoring when creating new-site
@vbaidak have you tried this locally? (I am pretty sure |
Hi. Agree. I'm not a Python guru I faced with this issue and this is the place that seems to be causing it. The problem is that None is passed as an argument instead of real db-name, which is provided. I don't know how to pass this variable from the annotation. May be you could help? |
@vbaidak try editing this file locally and check if this work. This certainly won't work |
@@ -22,7 +22,7 @@ def new_site(site, mariadb_root_username=None, mariadb_root_password=None, admin | |||
"Create a new site" | |||
frappe.init(site=site, new_site=True) | |||
|
|||
_new_site(None, site, mariadb_root_username=mariadb_root_username, mariadb_root_password=mariadb_root_password, admin_password=admin_password, | |||
_new_site(db-name, site, mariadb_root_username=mariadb_root_username, mariadb_root_password=mariadb_root_password, admin_password=admin_password, | |||
verbose=verbose, install_apps=install_app, source_sql=source_sql, force=force) |
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.
should be db_name
.
Fixed --db-name ignoring when creating new-site
Hi! Many thanks! I've checked it locally, it works now |
Fixed --db-name ignoring when creating new-site