NeteaseYanxuan / OSSA

Taro官方推荐的企业级多端 UI 库。目前已支持React语法。
https://ossa.miaode.com
MIT License
389 stars 33 forks source link

fix: 修复 upload 组件 max 和 multiple 属性不生效问题 #109

Closed kiner-tang closed 1 year ago

kiner-tang commented 1 year ago

这个 PR 做了什么? (简要描述所做更改)

关联ISSUE

这个 PR 是什么类型? (至少选择一个)

这个 PR 涉及以下平台:

changeset-bot[bot] commented 1 year ago

🦋 Changeset detected

Latest commit: 739e51fb02082c4c530bcf200017f1c0cfd9c226

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages | Name | Type | | --------- | ----- | | ossaui | Patch | | ossa-demo | Patch | | ossa-doc | Patch |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

vercel[bot] commented 1 year ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
ossa-demo ✅ Ready (Inspect) Visit Preview Dec 15, 2022 at 0:40AM (UTC)
1 Ignored Deployment | Name | Status | Preview | Updated | | :--- | :----- | :------ | :------ | | **ossa-doc** | ⬜️ Ignored ([Inspect](https://vercel.com/ossa/ossa-doc/ALPN1VpWbTZej4FWmJu87RuLzwFN)) | | Dec 15, 2022 at 0:40AM (UTC) |
CaedmonW commented 1 year ago

multiple应该是没用了,感觉可以去掉,仅用max来控制是不是更合理?

kiner-tang commented 1 year ago

multiple应该是没用了,感觉可以去掉,仅用max来控制是不是更合理?

是的,个人也觉得有点多余,不过直接去掉会出现broken change,我把multiple设置为过期属性吧