search

mindbox-cloud / ios-sdk

Other
6 stars 16 forks source link

MBX-3584: Implement `migrateShownInAppsIds` in `MigrationManger` #439

Closed justSmK closed 2 months ago

justSmK commented 2 months ago

[iOS] Перенести migrateShownInAppsIds в MigrationManager#3584

Обрати внимание на возможно довольно сложное для понимания условие isNeeded в миграции. Сделал для уменьшения числа кода формата

        if let oldShownInAppsIds = persistenceStorage.shownInAppsIds, !oldShownInAppsIds.isEmpty {
             return true
         } else {
             return false
         }

Не стал выбрасывать никаких ошибок в guard в run, так как она и так не стартанет по условию isNeeded. Также не стал никак её внутри логировать отдельно.