Flagsmith 6.3 should support newer versions of Dio (or pin to a lower Dio version)

未关闭 适合新手
#112 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
72/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
dart, flutter
领域
mobile-dev

调研方向

编译错误指向 lib/src/core/events/event_processor.dart 第 375 行附近,那里对 DioExceptionType 类型的 e.type 进行的 switch 缺少 transformTimeout 的分支。请阅读该 switch 及其周围的错误处理,然后与 dio 的 dio_exception.dart 中的 DioExceptionType 枚举进行对比。完成标准是:通过补充缺失的分支或添加 default 分支,使该包在 dio 5.10 及更高版本下分析无误,同时事件处理器对其他类型的现有行为保持不变。

由索引模型根据 Issue 内容生成。

描述

Version 5.10 of Dio added a new DioExceptionType type. Since flagsmith switch (e.type) to match cases, it needs to include the new types:

../../../.pub-cache/hosted/pub.dev/flagsmith-6.3.0/lib/src/core/events/event_processor.dart:375:15: Error: The type 'DioExceptionType' is not exhaustively matched by the switch cases since it doesn't match 'DioExceptionType.transformTimeout'.
 - 'DioExceptionType' is from 'package:dio/src/dio_exception.dart' ('../../../.pub-cache/hosted/pub.dev/dio-5.11.1/lib/src/dio_exception.dart').
Try adding a default case or cases that match 'DioExceptionType.transformTimeout'.
    switch (e.type) {
              ^

See also: https://github.com/cfug/dio/issues/2547#issuecomment-4841979338

Alternatively, pin dio to < 5.10.0
Alternatively alternatively, provide a default match.

主要语言
Dart
星标
17
派生
18
平均合并
3 天 16 小时
30 天内合并 PR
8

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

Flagsmith/flagsmith-flutter-client 的其他 Issue

查看 Flagsmith/flagsmith-flutter-client 的全部 Issue

相似的 Issue

更多 Dart Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。