Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Vexana not supported on Web Adapter. #115

Closed
sezginaliunal opened this issue Sep 8, 2024 · 9 comments
Closed

Vexana not supported on Web Adapter. #115

sezginaliunal opened this issue Sep 8, 2024 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@sezginaliunal
Copy link

Merhaba Veli Hocam. Aşağıdaki gibi bir base network sınıfı oluşturdum. Buradaki sorun isteklerimi Chrome üzerinden attığımda hata almam

Hata =>
TypeError: Cannot read properties of undefined (reading 'new')
packages/vexana/src/feature/adapter/web_adapter.dart 8:5 createAdapter
packages/vexana/src/mixin/core/network_manager_initialize.dart 16:33 [_setup]
packages/vexana/src/network_manager.dart 58:5 new
packages/minersy_blum/core/services/api/base_network.dart 13:22 new
packages/minersy_blum/app/services/user_service.dart 9:3 new
packages/minersy_blum/app/features/init/init_controller.dart 8:25 fetchUser


Kod =>
abstract class BaseNetwork<T extends INetworkModel> {
BaseNetwork({
required this.baseUrl,
bool isEnableLogger = false,
}) {
networkManager = NetworkManager(
isEnableLogger: isEnableLogger,
isEnableTest: true,

  interceptor: InterceptorsWrapper(
    onError: _onError,
  ),
  options: BaseOptions(
    baseUrl: baseUrl,
  ),
  // skippingSSLCertificate: true,
);

}

Future _onError(
DioException error,
ErrorInterceptorHandler handler,
) async {
if (error.response?.statusCode == null &&
await networkController.isConnected() == false) {}
handler.next(error);
}

final networkController = NetworkController();

final String baseUrl;
late final INetworkManager networkManager;
}


class UserService extends BaseNetwork {
UserService({
super.baseUrl = ApiUrls.baseUrl,
super.isEnableLogger,
});

Future<List?> fetchUser() async {
final response = await networkManager.send<Todo, List>(
ApiUrls().fetchUsers(),
parseModel: Todo(),
method: RequestType.GET,
);

return response.data;

}
}

@sezginaliunal
Copy link
Author

Not = Emülator veya gerçek cihazlarda herhangi bir sorun yaşanmamaktadır.

@serhatbekirak
Copy link

Bu sorunla ben de karşılaştım. Çözüm bulursan @sezginaliunal buraya yazmanı rica ediyorum

@VB10
Copy link
Owner

VB10 commented Sep 12, 2024

@serhatbekirak @sezginaliunal ben example altında olan projeyi çalıştırdım webde bir sorun göremedim ama acaba son versiyona mı geçmediniz vexana versiyonuuz kaçtır acaba

@VB10 VB10 self-assigned this Sep 12, 2024
@VB10 VB10 added the bug Something isn't working label Sep 12, 2024
@VB10
Copy link
Owner

VB10 commented Sep 12, 2024

burada olan kodu inceleyebilir chrome da run edebilirsiniz sorun göremedim ben
https://github.com/VB10/vexana/tree/master/example

@kurtali7
Copy link

kurtali7 commented Sep 12, 2024

@VB10 bende aynı sorunu yaşadım. Sorun sadece webde oluyor bu arada native de sorun yok

Flutter : 3.24.2
vexana : 5.0.0

Araştırdığımda dio_web_adapter paketinin güncellenmesi gerekir sanırım. Ama geçici çözüm olarak sen kütüphaneninde şunu fixleyebilirsin;

web_adapter.dart sınıfı aşağıdaki şekilde düzenlendiğinde çalışıyor;

HttpClientAdapter createAdapter({bool isEnableTest = false}) {
final adapter = HttpClientAdapter() as BrowserHttpClientAdapter;
adapter.withCredentials = true;
return adapter;
}

@VB10
Copy link
Owner

VB10 commented Sep 12, 2024

@kurtali7 evet versiyon yukselttim dio web adaptor hata veriyor, cozum olarak senin de ekledigin ana repoda olan bunu gordum bende bir bakip paket gonderecegim tesekkurler
cfug/dio#2282 (comment)

@VB10
Copy link
Owner

VB10 commented Sep 12, 2024

yeni paket gönderiyorum 5.0.1 olarak bu issue çözüldüğünde bu geçici çözümü geri alacağım herkese teşekkürler

@sezginaliunal
Copy link
Author

Herkese teşekkür ederim gerekli testleri yapacağım
Hocam sağolun.

VB10 added a commit that referenced this issue Sep 12, 2024
Update vexana version to 5.0.1 and dio version to 5.7.0 Vexana not su…
@VB10
Copy link
Owner

VB10 commented Sep 12, 2024

5.0.1 olarak gönderdim paketi herkese kolay gelsin elinize sağlık tekrardan <3

@VB10 VB10 closed this as completed Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants