From 2e28ee5b25ad407d178b19b7efd1a2943603f073 Mon Sep 17 00:00:00 2001 From: Gavin D'souza Date: Mon, 15 Nov 2021 12:10:50 +0530 Subject: [PATCH 1/2] fix: Update Installed Applications on remove_app Removing an app from a site should sync the Installed Applications doctype --- frappe/installer.py | 1 + 1 file changed, 1 insertion(+) diff --git a/frappe/installer.py b/frappe/installer.py index d1a13fdaab..9eed44ea15 100755 --- a/frappe/installer.py +++ b/frappe/installer.py @@ -240,6 +240,7 @@ def remove_app(app_name, dry_run=False, yes=False, no_backup=False, force=False) if not dry_run: remove_from_installed_apps(app_name) + frappe.get_single('Installed Applications').update_versions() frappe.db.commit() click.secho(f"Uninstalled App {app_name} from Site {site}", fg="green") From ef39843f6ba8b24912fdb41a62df186dbb39bd15 Mon Sep 17 00:00:00 2001 From: Gavin D'souza Date: Mon, 15 Nov 2021 13:32:34 +0530 Subject: [PATCH 2/2] fix(cli): Add new line after each site's migrate --- frappe/commands/site.py | 1 + 1 file changed, 1 insertion(+) diff --git a/frappe/commands/site.py b/frappe/commands/site.py index c5f78e2680..3c7f2f5525 100755 --- a/frappe/commands/site.py +++ b/frappe/commands/site.py @@ -461,6 +461,7 @@ def migrate(context, skip_failing=False, skip_search_index=False): skip_search_index=skip_search_index ) finally: + print() frappe.destroy() if not context.sites: raise SiteNotSpecifiedError