Adaptive target refactoring #229
Conversation
| } | ||
|
|
||
| override func refresh() { | ||
| lastSolvedDay = statsManager.getLastDays(count: 1)[0] > 0 ? statsManager.dayByDate(Date()) : 0 |
There was a problem hiding this comment.
а почему бы это тоже в statsManager не вынести?
|
|
||
| func application(_ application: UIApplication, open url: URL, sourceApplication: String?, annotation: Any) -> Bool { | ||
| print("opened app via url \(url.absoluteString)") | ||
| if VKSdk.processOpen(url, fromApplication: sourceApplication) { |
There was a problem hiding this comment.
выпилили, потому что до этого забыли?
There was a problem hiding this comment.
Да, в какой-то момент авторизацию выпилили, а вот в делегате этот код нет.
|
@Ostrenkiy посмотри ещё раз. |
| var loadingDoneCallback: (() -> Void)? | ||
| var fetchComplete: Bool = false | ||
|
|
||
| lazy var appDefaults: [String: NSObject] = [ |
There was a problem hiding this comment.
А для чего это выносить в класс?
There was a problem hiding this comment.
Потому что в адаптивных RemoteConfig нужно расширить, добавив новый параметр. Для этого нужно перезадавать дефолтные значения, потому что просто добавить дефолтное значение для одного нового ключа нельзя.
|
|
||
| extension AnalyticsEvents { | ||
| struct Adaptive { | ||
| struct AdaptiveApp { |
There was a problem hiding this comment.
А почему мы хотим выпилить все эти ивенты?
There was a problem hiding this comment.
Давай добавим adaptive_submission_created ивент?
There was a problem hiding this comment.
Выпилили только adaptive_achievement_level, adaptive_achievement_achievement и adaptive_achievement_share_clicked. Они нужны?
Остальные ивенты просто убраны отсюда в AnalyticsEvents.swift
Задача: #APPS-1757
Коротко для Release Notes, в формате «Сделали/Добавили/Исправили N»:
Актуализировали адаптивные таргеты
Описание:
Код из адаптивных таргетов перенесен в основной (в #213 он был скопирован). Теперь в адаптивных переиспользуется с наследованием код из основного приложения.
Значение supportedCourses в конфиге – для адаптивных таргетов имеется возможность задавать несколько курсов (будет использован экран выбора).
TODO
Исправить конфиги для всех таргетов