[Do Not Merge] MBX-3751: WebView Example #475
Conversation
Example/Example/AppDelegate.swift
Outdated
| endpoint: "Mpush-test.ReleaseExample.IosApp", | ||
| domain: "api.mindbox.ru", | ||
| endpoint: "Test-staging.Test01", | ||
| domain: "api-staging.mindbox.ru", |
There was a problem hiding this comment.
дававй тут обратно вернем на эндпоинт экзампла
|
Ветка в инструкциях https://github.com/mindbox-cloud/ios-sdk/tree/webView_integration |
|
Я бы тут предложил сделать что-нибудь типа example/MBX-Bla-bla-bla |
Я сделал копирку с Android, Flutter. Тогда надо всё переименовывать. |
| func syncMindboxDeviceUUIDs() { | ||
| Mindbox.shared.getDeviceUUID { [weak webView] uuid in | ||
| guard let webView, !uuid.isEmpty else { | ||
| Logger.common(message: "[WebView]: Device UUID is empty or invalid", level: .error) |
There was a problem hiding this comment.
У нас есть публичный метод
Mindbox.logger.log
Импорт логгера не нужен
|
|
||
| @Observable final class ViewModel { | ||
|
|
||
| weak var webView: WKWebView? |
There was a problem hiding this comment.
Насколько логично докапываться до тестового функционала?)
Как будто совсем не логично)
просто для инфы
webView во viewModel рушит логику MVVM
There was a problem hiding this comment.
А если переделать, то как лучше? Через delegate? Или прям в каждый метод передавать ссылку на webView напрямую и не хранить его во ViewModel?
|
Да, переименуйте везде. Потому что мы потом забудем и снесем ее нафиг. А одинаковые названия веток в example нас на что-нибудь натолкнут |
|
Убрал лишний метод, поменял логгер, чутка изменил ViewModel, вынес WKNavigationDelegate в расширение для удобства. |
[iOS]: Example для интеграции WebView#3751