WMSDK-411: Rename config JSON key #502
Conversation
Add tests
| private func saveInappSessionToCache(inappSession: String?) { | ||
| SessionTemporaryStorage.shared.expiredInappSession = inappSession | ||
| Logger.common(message: "Saved slidingExpiration.inappSession - \(inappSession) to temporary storage.") | ||
| private func saveInappSessionToCache(config: String?) { |
There was a problem hiding this comment.
Может тут переименовать название метода? saveMobileConfigSessionToCache к примеру
Метод getInappSession тоже может стоит переименовать?
There was a problem hiding this comment.
А как лучше поступить с InappSessionManager?
Думаю класс оставить называться также, расположение то же в InAppMessages, внутренние методы переименовать на updateInappSession, hideInappIfConfigSessionExpired, getConfigSession, logNearestConfigSessionExpirationTime.
А протокольный метод checkInappSession оставить как есть? Или тоже переименовать на checkConfigSession? Сейчас его вызов выглядит так inappSessionManager.checkInappSession().
There was a problem hiding this comment.
Я бы не переименовывал InappSessionManager, так как он действительно отвечает исключительно за сессию инаппов.
* WMSDK-411: Rename config JSON key Add tests Renaming some properties and functions
[iOS]: Поддержать изменение контракта в секции slidingExpiration
Previous Pull Request
Основные изменения в
Mindbox/InAppMessages/Configuration/InAppConfigurationManager.swiftMindbox/InAppMessages/Models/Config/SlidingExpirationModel.swiftОстальное в конфигах тестов и самих тестах. Где-то добавил проверки парсинга.