Skip to content

Conversation

@flora-gr
Copy link

The changes in this branch fix a compilation issue in our iOS builds.
Without this, we got the following errors:

❌ /Users/runner/hostedtoolcache/Flutter/3.0.3/macos/flutter/.pub-cache/hosted/pub.dartlang.org/webview_pro_wkwebview-2.7.1+2/ios/Classes/FlutterWebView.m:11:17: definition of 'FLTWebViewFactory' must be imported from module 'webview_flutter_wkwebview.FlutterWebView' before it is required

@implementation FLTWebViewFactory {
^

❌ /Users/runner/hostedtoolcache/Flutter/3.0.3/macos/flutter/.pub-cache/hosted/pub.dartlang.org/webview_pro_wkwebview-2.7.1+2/ios/Classes/FlutterWebView.m:120:29: definition of 'FLTWKNavigationDelegate' must be imported from module 'webview_flutter_wkwebview.FLTWKNavigationDelegate' before it is required

_navigationDelegate = [[FLTWKNavigationDelegate alloc] initWithChannel:_channel];
       ^

❌ /Users/runner/hostedtoolcache/Flutter/3.0.3/macos/flutter/.pub-cache/hosted/pub.dartlang.org/webview_pro_wkwebview-2.7.1+2/ios/Classes/FlutterWebView.m:153:6: definition of 'FLTWKProgressionDelegate' must be imported from module 'webview_flutter_wkwebview.FLTWKProgressionDelegate' before it is required

[_progressionDelegate stopObservingProgress:_webView];
       ^

▸ Compiling FLTWebViewFlutterPlugin.m
▸ Compiling FLTWKProgressionDelegate.m
▸ Compiling FLTWKNavigationDelegate.m
** ARCHIVE FAILED **

@thdwlsgus0
Copy link

thdwlsgus0 commented Sep 9, 2022

@wenzhiming
I am also experiencing the same thing. If this error is resolved, it is likely that the build will be done in the ios webview environment. Is it possible to edit?

스크린샷 2022-09-09 오후 10 05 52

please merge bb..

@ivangoluuk
Copy link

i have this error too

Definition of 'FLTCookieManager' must be imported from module 'webview_flutter_wkwebview.FLTCookieManager' before it is required

@ivangoluuk
Copy link

please fix this issue :(

@wenzhiming wenzhiming merged commit bc3fc2a into wenzhiming:dev Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants