Repository: RasaHQ/rasa
Pull request: Improve success message of data migration
closes #10242

**Proposed changes**:
This improves the success message when domain data is successfully migrated from 2.0 to 3.0



Problem statement:
`rasa data migrate` should give more informative message
Upon successfully running `rasa data migrate`, I am told:
```
Your domain was successfully migrated! It's location is domain.yml.
```

I think this isn't clear enough and we should change it to something like:
```
Your domain file (<original domain name file/path>) was successfully migrated! The migrated version is now <new file/path>. The original domain file is backed-up as <back-up file/path>.
```
Note: The message is this brief even if I also had some file named `original_domain.yml` (by accident) which got overwritten by the migration command.

Hints from issue discussion:

Commit message: address comment

Patch to review:
diff --git a/rasa/core/migrate.py b/rasa/core/migrate.py
index 65a22028b434..8fad48e9c241 100644
--- a/rasa/core/migrate.py
+++ b/rasa/core/migrate.py
@@ -245,6 +245,7 @@ def migrate_domain_format(
     if domain_file.is_dir():
         backup_dir = current_dir / "original_domain"
         backup_dir.mkdir()
+        backup_path = backup_dir / "original_domain.yml"
 
         if out_file.is_file() or not out_file.exists():
             out_file = current_dir / "new_domain"
@@ -274,8 +275,8 @@ def migrate_domain_format(
                 f"domain file. Only domain yaml files can be migrated. "
             )
 
-        backup = current_dir / "original_domain.yml"
-        original_domain = _create_back_up(domain_file, backup)
+        backup_path = current_dir / "original_domain.yml"
+        original_domain = _create_back_up(domain_file, backup_path)
 
     new_forms, updated_slots = _migrate_form_slots(original_domain)
     new_slots = _migrate_auto_fill_and_custom_slots(original_domain, updated_slots)
@@ -283,5 +284,7 @@ def migrate_domain_format(
     _write_final_domain(domain_file, new_forms, new_slots, out_file)
 
     rasa.shared.utils.cli.print_success(
-        f"Your domain was successfully migrated! It's location is {out_file}."
+        f"Your domain file '{str(domain_file)}' was successfully migrated! "
+        f"The migrated version is now '{str(out_file)}'. "
+        f"The original domain file is backed-up as '{str(backup_path)}'."
     )


Code review comments:
diff_hunk: @@ -245,6 +245,7 @@ def migrate_domain_format(
     if domain_file.is_dir():
         backup_dir = current_dir / "original_domain"
         backup_dir.mkdir()
+        backup_path = backup_dir / "original_domain.yml"
path: rasa/core/migrate.py
text: @user1:
Oh this line is not necessary, you can rename `backup_dir` to `backup_path`, that's what i meant.

@author:
Okay - so the message will contain either a path to a directory, or a path to a file.

@user1:
Yup, exactly!