activeviam / atoti-ui-migration

Migrate content saved with old versions of ActiveUI

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Better feedback for migration results

ibolari42 opened this issue · comments

Hi,

Its great we have this to cut down the manual effort involved in the AUI migration but I think there's some room for improvement in the feedback department.

At the moment, there are error messages logged to the terminal if a dashboard cannot be migrated but this approach does not scale well. I'm currently migrating a project with over 2000 dashboards and with over 300 failures you can imagine that the error messages get lost eventually. A better approach could be a .log file containing all the relevant errors for later review.

It also isn't clear when the tool is done. after logging all the errors it just pauses for a while (the run doesn't terminate) and I have to assume it is done because the output file exists... this isn't great if I already had an output file in my folder from a previous run that I'm expecting the tool to overwrite. a simple done, you can now exit logged to the terminal should do.

Lastly a high level summary of the results would also be great. you can probably put in something more robust than I've done below as I'm just guessing at the general structure of the json files, so I'm probably missing something but an output like this would be good.

Thanks in advance 👍 .

{
  "migrationResults": {
    "AUI4": {
      "filters": 65,
      "dashboards": {
        "total": 2413,
        "pages": 3406,
        "widgets": 3769
      },
      "savedWidgets": 116
    },
    "AUI5": {
      "filters": 65,
      "dashboards": {
        "total": 2112,
        "pages": 2588,
        "widgets": 2826
      },
      "savedWidgets": 112
    },
    "Diff": {
      "filters": 0,
      "dashboards": {
        "total": 301,
        "pages": 818,
        "widgets": 943
      },
      "savedWidgets": 4,
      "unSupportedWidgets": {
        "total": 140,
        "filters": {
          "total": 5,
          "affected": [
            "'Page Filters' in page 'REG Standard VaR - PF tree vs LE tree' of dashboard b273: REG Standard VaR LE",
            "'Page Filters' in page 'Page 1' of dashboard 63e: PF - VaR - Vector",
            "'Page Filters' in page 'Page 1' of dashboard 5b9: PF - VaR - Comp",
            "'Page Filters' in page 'Page 1' of dashboard 7bb: PV and Notional 2",
            "'Page Filters' in page 'Page 1' of dashboard d3f: TSY - VaR - Vector"
          ]
        },
        "context-values": {
          "total": 135,
          "affected": [
            "'Page Context Values' in page 'INT Standard VaR Vector Analysis' of dashboard 113: INT Standard VaR Scenario Analysis_LuxBranch",
            "'Context Values' in page 'Page 1' of dashboard 177: Comparison INT Global Stress (Flexible Stress) Branch",
            "'Context Values' in page 'CS, IR, FX separated' of dashboard 185: Comparison INT Sensitivities",...
          ]
        }
      }
    }
  }
}

output generated from
https://github.com/activeviam/IBolari-AUI5-customisations/blob/mrex-ui-migration/migrationAnalyser.js

resolved by #33