[BUG] dcc.Upload contents and dcc.Geolocation position docstrings describe the wrong value format

未关闭
#4,125 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

@ZainnQureshii 已经在做这个了。

开始于 2026年11月6日。

  • #4126 来自 @ZainnQureshii —— 未关闭

评估

难度
1/5
预计耗时
1 小时以内
新手友好度
30/100
Issue 类型
文档
描述清晰度
描述清楚
活跃度
停滞
技术栈
python, react

调研方向

Update the dcc.Upload.contents and dcc.Geolocation.position docstrings in the cited component files: components/dash-core-components/src/components/Upload.react.js and src/components/Geolocation.react.js. Read the current wording alongside the behavior cited in the issue, then check the generated Python help for both components. Done when the help describes contents as a base64 data URL and unavailable position fields as null (None in Python); pull request #4126 is already open.

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

描述

bug P3 size: 1

Describe your context

Current dev (dash 4.5.0). Docs only; no specific browser or OS.

Describe the bug

Two dcc prop docstrings describe a value format the component does not produce. Both reach users through the generated Python help (help(dcc.Upload), help(dcc.Geolocation)).

  1. dcc.Upload.contents says "The contents of the uploaded file as a binary string" (components/dash-core-components/src/components/Upload.react.js:29). The component reads every file with reader.readAsDataURL(file) (src/fragments/Upload.react.js:189) and passes reader.result through unchanged (:173), so contents is a base64 data URL (data:<mime type>;base64,<data>), which callbacks have to split and base64-decode.

  2. dcc.Geolocation.position says the optional fields are included "when available, otherwise it will be NaN" (src/components/Geolocation.react.js:122). The component sets each field with ?? null (:59-65), so an unavailable value arrives as null (None in a Python callback), not NaN.

Expected behavior

  • contents: "The contents of the uploaded file as a base64 encoded data URL (data:<mime type>;base64,<data>)".
  • position: "... otherwise it will be null (None in Python)".

I'm happy to send the PR for both lines.

(Found and written up with AI assistance, Claude Code; the cited lines were checked against dev.)

主要语言
Python
星标
24.5k
派生
2.3k
平均合并
3 天 7 小时
30 天内合并 PR
30

环境准备

从这里开始

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

plotly/dash 的其他 Issue

查看 plotly/dash 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

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