Skip to content

Feature/mobilewebview 23 web view bridge#641

Merged
Vailence merged 25 commits intomission/webView-inAppfrom
feature/MOBILEWEBVIEW-23-WebViewBridge
Feb 3, 2026
Merged

Feature/mobilewebview 23 web view bridge#641
Vailence merged 25 commits intomission/webView-inAppfrom
feature/MOBILEWEBVIEW-23-WebViewBridge

Conversation

@Vailence Vailence requested a review from enotniy January 28, 2026 10:24
@Vailence Vailence changed the title [WIP] Feature/mobilewebview 23 web view bridge Feature/mobilewebview 23 web view bridge Jan 29, 2026
@Vailence Vailence requested a review from justSmK January 29, 2026 11:49
@Vailence Vailence force-pushed the feature/MOBILEWEBVIEW-23-WebViewBridge branch from 6f6ddbd to 3fa303c Compare February 2, 2026 11:56
)

// TODO: - Тут какая то фигня, с текущей страницей цикл происходит запросов, request - response. Пока закомментировал
// bridge.send(response)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а как тогда работает ready?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Да, надо раскомментить это. Я вчера проверял, работало корректно вроде
Надо чтобы Акыл тоже явно чекнул это


if message.type == .request {
pendingRequestIds.insert(message.id)
} else if message.type == .response {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а .error ?

@Vailence Vailence merged commit 6cf2edf into mission/webView-inApp Feb 3, 2026
4 checks passed
@Vailence Vailence deleted the feature/MOBILEWEBVIEW-23-WebViewBridge branch February 3, 2026 11:32
justSmK pushed a commit that referenced this pull request Feb 6, 2026
* WMSDK-Bridge Message

* WMSDK- Move down extensions in WebViewController

* WMSDK-Added WebView + JS Bridge

* WMSDK-Simple variation

* WMSDK-Second iteration

* WMSDK-Third iteration

all good yet

* WMSDK-Old script from Native side

* WMSDK-Added customPage

* WMSDK-Updated error message

* WMSDK-Added direct BridgeMessage delegate

* WMSDK-Update id save/remove

* WMSDK-0000 Refactor code

* WMSDK-0000 Bridge Message dispatcher

* WMSDK-0000 Remove custom JS from SDK

* WMSDK-0000 Remove unused MindboxWebView

* WMSDK-0000 Added protocol

* WMSDK-0000 Update test html (remove later)

* WMSDK-0000 Fix warnings

* WMSDK-0000 Add start method after init

* WMSDK-0000 Ready event

* WMSDK-0000 Fix PR Comments

* WMSDK-0000 Added TODOs for future generations

* WMSDK-0000 Fix linter

* WMSDK-0000 Uncomment bridge.send

* WMSDK-0000 Updated log about SDKBridge missing

---------

Co-authored-by: Vailence <utekeshev@mindbox.cloud>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants