fix: migrate log for software profiles (#72)

Co-authored-by: Begoña Álvarez de la Cruz <[email protected]>

Author: cpl121

packages/desktop/electron/electronApi.js

@@ -118,45 +118,12 @@ const ElectronApi = {
      *
      * @method exportMigrationLog
      *
-     * @param {string} sourcePath
-     * @param {string} defaultFileName
-     *
-     * @returns {Promise<boolean>}
-     */
-    exportMigrationLog: (sourcePath, defaultFileName) =>
-        ipcRenderer
-            .invoke('show-save-dialog', {
-                properties: ['createDirectory', 'showOverwriteConfirmation'],
-                defaultPath: defaultFileName,
-                filters: [{ name: 'Log Files', extensions: ['log'] }],
-            })
-            .then((result) => {
-                if (result.canceled) {
-                    return null
-                }
-
-                return new Promise((resolve, reject) => {
-                    fs.copyFile(sourcePath, result.filePath, (err) => {
-                        if (err) {
-                            reject(err)
-                        } else {
-                            resolve(true)
-                        }
-                    })
-                })
-            }),
-
-    /**
-     * Exports ledger migration log
-     *
-     * @method exportLedgerMigrationLog
-     *
      * @param {string} content
      * @param {string} defaultFileName
      *
      * @returns {Promise}
      */
-    exportLedgerMigrationLog: (content, defaultFileName) =>
+    exportMigrationLog: (content, defaultFileName) =>
         ipcRenderer
             .invoke('show-save-dialog', {
                 properties: ['createDirectory', 'showOverwriteConfirmation'],

packages/shared/lib/tests/mocks/platform.ts

@@ -9,10 +9,7 @@ const Platform: IPlatform = {
     PincodeManager: undefined,
     ledger: undefined,
     close(): void {},
-    exportLedgerMigrationLog(content: unknown, defaultFileName: string): Promise<boolean | null> {
-        return Promise.resolve(undefined)
-    },
-    exportMigrationLog(sourcePath: string, defaultFileName: string): Promise<boolean | null> {
+    exportMigrationLog(content: unknown, defaultFileName: string): Promise<boolean | null> {
         return Promise.resolve(undefined)
     },
     exportTransactionHistory(defaultPath: string, contents: string): Promise<string | null> {

packages/shared/lib/typings/platform.ts

@@ -15,8 +15,7 @@ export interface IPlatform {
     getStrongholdBackupDestination(defaultPath: string): Promise<string | null>
     saveStrongholdBackup({ allowAccess }: { allowAccess: boolean }): Promise<void>
     exportTransactionHistory(defaultPath: string, contents: string): Promise<string | null>
-    exportMigrationLog(sourcePath: string, defaultFileName: string): Promise<boolean | null>
-    exportLedgerMigrationLog(content: unknown, defaultFileName: string): Promise<boolean | null>
+    exportMigrationLog(content: unknown, defaultFileName: string): Promise<boolean | null>
     getUserDataPath(): Promise<string>
     getDiagnostics(): Promise<{ label: string; value: string }[]>
     getOS(): Promise<string> | string

packages/shared/routes/setup/Congratulations.svelte

@@ -56,11 +56,7 @@
     function exportMigrationLog(): void {
         exportMigrationLogBusy = true
         getProfileDataPath($activeProfile.id)
-            .then((source) =>
-                $walletSetupType === SetupType.TrinityLedger
-                    ? Platform.exportLedgerMigrationLog($migrationLog, `${$activeProfile.id}-${LOG_FILE_NAME}`)
-                    : Platform.exportMigrationLog(`${source}/${LOG_FILE_NAME}`, `${$activeProfile.id}-${LOG_FILE_NAME}`)
-            )
+            .then(() => Platform.exportMigrationLog($migrationLog, `${$activeProfile.id}-${LOG_FILE_NAME}`))
             .catch((error) => {
                 console.error(error)
                 showAppNotification({