Skip to content

fix: replace error dialogs with OKToast and align image upload timeouts - #208

Merged
3316891527 merged 1 commit into
devfrom
fix/error-toasts-and-draw-timeouts
Nov 14, 2026
Merged

3316891527 merged 1 commit into
devfrom
fix/error-toasts-and-draw-timeouts

Conversation

@3316891527

Copy link
Copy Markdown
Owner

变更说明 / Description

背景与动机 / Context and motivation

包管理导入失败仍弹出截图中的“请求失败”模态窗口;同类错误提示还分散在其他页面。MiniMax、Nanobanana 的参考图上传也没有传入宿主超时,仍使用默认读 20 秒、写 15 秒。

改动范围 / Changes

  • 扫描 211 个含对话框调用的 Android 文件,迁移 7 个纯错误展示入口:包管理、聊天、终端、聊天文件夹、ToolPkg Compose DSL 对话框宿主、应用更新检查、市场更新检查。
  • 全部使用普通 OKToast.error,不新增 requestError 调用;Compose 入口发出提示后消费一次错误状态。
  • 删除 ErrorDialog.kt、其全部引用及旧的 dismissErrorDialog 命名。
  • 市场无需后续操作的检查结果改用对应状态 Toast,保留有更新时的详情跳转确认。
  • MiniMax、Nanobanana 和独立 BeeIMG 上传器显式传入 600000 毫秒,TS 和分发 JS 同步。
  • 核对其余生图脚本:OpenAI 已修复;Qwen、SiliconFlow、SpaceXAI、Zhipu 的请求使用自建 600 秒客户端。
  • 审计详情记录于 docs/TODO/fix/error-toast-and-draw-timeouts/index.md。

兼容性与风险 / Compatibility and risks

提示改为非模态,需要随新版 APK 生效。保留确认、编辑、下载进度和用户主动打开的错误详情界面。脚本沿用既有 API 地址、表单及响应解析,只延长此前受宿主短超时限制的上传请求;运行参数和数据格式保持兼容。

关联 Issue / Related issue

N/A:维护者要求统一截图同类错误提示,并排查同源生图脚本的上传超时。

验证方式 / Verification

git diff --check:通过。
node --check:minimax_draw.js、nanobanana_draw.js、beeimg_image_uploader.js 均通过。
全应用源码检索:无 ErrorDialog 或 dismissErrorDialog 残留引用。
本次新增错误展示:全部为普通 OKToast.error。
未运行本地 Gradle,Android 编译和 JVM 检查由本 PR 的 GitHub Actions 执行。
未调用收费生图服务或使用真实 API Key 发送生成请求。

证据 / Evidence

修改计数和完整脚本审计见本 PR 附带的审计文档。Android 界面验证以远端 CI 和后续设备验证为准,不将静态审计当作设备实测。

检查清单 / Checklist

  • 已记录验证及未运行项原因。
  • 日常开发 PR 目标为 dev。
  • 已确认 Candidate checks 将覆盖适用范围,并会处理技术失败。
  • 改动仅包含错误提示迁移、上传超时及对应审计记录。
  • TS/JS 同步,沿用已支持的 timeout_ms 参数,未改变接口协议。

@3316891527
3316891527 merged commit 8e8905e into dev Nov 14, 2026
5 of 6 checks passed
@3316891527
3316891527 deleted the fix/error-toasts-and-draw-timeouts branch November 14, 2026 10:04
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.

1 participant