fix: migration log (#65)

* feat: continue with the flow of migration

* feat: add prepareMigrationLog tp send migration request

* fix: debris

* fix: error creating a second profile

* fix: handle error

* fix: handle error with software profile

---------

Co-authored-by: Branko Bosnic <[email protected]>

Author: cpl121

packages/shared/lib/migration.ts

@@ -372,7 +372,7 @@ export const prepareMigrationLog = (bundleHash: string, trytes: string[], balanc
     const { bundles } = get(migration)
 
     const bundle = get(bundles).find((bundle) => bundle.bundleHash === bundleHash)
-    const spentInputs = bundle.inputs.filter((input) => input.spent === true)
+    const spentInputs = bundle?.inputs?.filter((input) => input.spent === true) || []
 
     const spentBundleHashes = []
 
@@ -391,9 +391,9 @@ export const prepareMigrationLog = (bundleHash: string, trytes: string[], balanc
             receiveAddressTrytes: transactionObjects.find((tx) => tx.address.startsWith('TRANSFER')).address,
             balance,
             spentBundleHashes,
-            spentAddresses: bundle.inputs.filter((input) => input.spent === true).map((input) => input.address),
-            mine: bundle.miningRuns > 0,
-            crackability: bundle.crackability,
+            spentAddresses: bundle?.inputs?.filter((input) => input.spent === true).map((input) => input.address) || [],
+            mine: bundle?.miningRuns > 0,
+            crackability: bundle?.crackability || null,
         },
     ])
 }
@@ -730,14 +730,10 @@ export const sendLedgerMigrationBundle = (bundleHash: string, trytes: string[]):
  * @returns {Promise<Receipt>}
  */
 export const sendOffLedgerMigrationRequest = async (trytes: string[]): Promise<any> => {
-    try {
-        const offLedgerHexRequest = createOffLedgerRequest(trytes)
-        await fetchOffLedgerRequest(offLedgerHexRequest.request)
-        return await fetchReceiptForRequest(offLedgerHexRequest.requestId)
-    } catch (err) {
-        showAppNotification({ type: 'error', message: err.message || 'Failed to send migration request' })
-        return
-    }
+    const offLedgerHexRequest = createOffLedgerRequest(trytes)
+    await fetchOffLedgerRequest(offLedgerHexRequest.request)
+    const receiptResponse = await fetchReceiptForRequest(offLedgerHexRequest.requestId)
+    return receiptResponse
 }
 /**
  * Creates migration bundle

packages/shared/routes/setup/Welcome.svelte

@@ -16,7 +16,7 @@
     import { showAppNotification } from 'shared/lib/notifications'
     import { Platform } from 'shared/lib/platform'
     import { openPopup } from 'shared/lib/popup'
-    import { newProfile, storeProfile, validateProfileName } from 'shared/lib/profile'
+    import { newProfile, profiles, storeProfile, validateProfileName } from 'shared/lib/profile'
     import { Stage } from 'shared/lib/typings/stage'
     import { destroyActor, getProfileDataPath, initialise } from 'shared/lib/wallet'
     import { get } from 'svelte/store'
@@ -32,7 +32,9 @@
     $: hasDeveloperProfileChanged = $newProfile?.isDeveloperProfile !== isDeveloperProfile
 
     async function handleContinueClick(): Promise<void> {
-        const trimmedProfileName = TMP_PROFILE_NAME.trim()
+        const _profiles = get(profiles)
+        const trimmedProfileName = TMP_PROFILE_NAME.trim() + _profiles.length
+
         try {
             validateProfileName(trimmedProfileName)
         } catch (err) {

packages/shared/routes/setup/migrate/views/Migrate.svelte

@@ -12,6 +12,7 @@
         hasBundlesWithSpentAddresses,
         hasSingleBundle,
         migration,
+        prepareMigrationLog,
         removeAddressChecksum,
         sendOffLedgerMigrationRequest,
         unselectedInputs,
@@ -29,7 +30,6 @@
     import { MigrationAddress } from '@lib/typings/migration'
     import { createPrepareTransfers } from '@iota/core'
     import { appRouter } from '@core/router'
-    import { convertBech32AddressToEd25519Address } from '@lib/ed25519'
 
     export let locale: Locale
 
@@ -88,17 +88,13 @@
             balance: input.balance,
         }))
 
-        try {
-            const bundleTrytes: string[] = await prepareTransfers(get(seed), transfers, {
-                inputs: inputsForTransfer,
-            })
+        const bundleTrytes: string[] = await prepareTransfers(get(seed), transfers, {
+            inputs: inputsForTransfer,
+        })
 
-            return sendOffLedgerMigrationRequest(bundleTrytes.reverse())
-        } catch (err) {
-            showAppNotification({ type: 'error', message: err.message || 'Failed to prepare transfers' })
-        } finally {
-            loading = false
-        }
+        // TODO: Check the bundlehash with software profiles
+        prepareMigrationLog('', bundleTrytes.reverse(), migratableBalance)
+        return sendOffLedgerMigrationRequest(bundleTrytes.reverse())
     }
 
     function handleContinueClick() {
@@ -116,6 +112,7 @@
                         .then(({ trytes, bundleHash }) => {
                             closePopup(true) // close transaction popup
                             singleMigrationBundleHash = bundleHash
+                            prepareMigrationLog(bundleHash, trytes.reverse(), migratableBalance)
                             return sendOffLedgerMigrationRequest(trytes.reverse())
                         })
                         .then((receipt) => {