From 791ac440375515b6ca4a1c5d53c2f1529bd9b47d Mon Sep 17 00:00:00 2001
From: Alexandre Aubin <alex.aubin@mailoo.org>
Date: Thu, 6 Dec 2018 22:10:06 +0100
Subject: [PATCH] Add success message after running migrations

---
 locales/en.json       | 1 +
 src/yunohost/tools.py | 3 +++
 2 files changed, 4 insertions(+)

diff --git a/locales/en.json b/locales/en.json
index 6ce22ca80..3bb36ae9d 100644
--- a/locales/en.json
+++ b/locales/en.json
@@ -305,6 +305,7 @@
     "migrations_show_currently_running_migration": "Running migration {number} {name}...",
     "migrations_show_last_migration": "Last ran migration is {}",
     "migrations_skip_migration": "Skipping migration {number} {name}...",
+    "migrations_success": "Successfully ran migration {number} {name}!",
     "migrations_to_be_ran_manually":  "Migration {number} {name} has to be ran manually. Please go to Tools > Migrations on the webadmin, or run `yunohost tools migrations migrate`.",
     "migrations_need_to_accept_disclaimer": "To run the migration {number} {name}, your must accept the following disclaimer:\n---\n{disclaimer}\n---\nIf you accept to run the migration, please re-run the command with the option --accept-disclaimer.",
     "monitor_disabled": "The server monitoring has been disabled",
diff --git a/src/yunohost/tools.py b/src/yunohost/tools.py
index f6535d85c..fbadb3b20 100644
--- a/src/yunohost/tools.py
+++ b/src/yunohost/tools.py
@@ -924,6 +924,9 @@ def tools_migrations_migrate(target=None, skip=False, auto=False, accept_disclai
                 logger.error(msg, exc_info=1)
                 operation_logger.error(msg)
                 break
+            else:
+                logger.success(m18n.n('migrations_success',
+                                      number=migration.number, name=migration.name))
 
         else:  # if skip
             logger.warn(m18n.n('migrations_skip_migration',