Skip to content

Conversation

Dolov
Copy link
Contributor

@Dolov Dolov commented Jan 28, 2021

No description provided.

@vercel
Copy link

vercel bot commented Jan 28, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/react-component/select/3tgd7tsye
✅ Preview: https://select-git-fork-dolov-master.react-component.vercel.app

@codecov
Copy link

codecov bot commented Jan 30, 2021

Codecov Report

Merging #590 (ee53e1c) into master (a15a8ad) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #590   +/-   ##
=======================================
  Coverage   99.27%   99.27%           
=======================================
  Files          20       20           
  Lines         967      971    +4     
  Branches      322      324    +2     
=======================================
+ Hits          960      964    +4     
  Misses          6        6           
  Partials        1        1           
Impacted Files Coverage Δ
src/utils/commonUtil.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a15a8ad...ee53e1c. Read the comment docs.

return finalVal;
});
rawValues = (values as LabelValueType[])
.filter(item => !!item)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

加个用例?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

加过啦 ~~

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~ 这个能合么

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

value 为 0 时,也被 filter 了

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Dolov 再来个 PR

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

value 为 0 时,也被 filter 了

labelInValue 模式下,需要支持这种非法数据格式么

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

有 ts 定义,先不加了

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

使用js的项目呢?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😂😂😂 对,忘了 还有 js

@afc163 afc163 merged commit 85372b2 into react-component:master Mar 31, 2021
ceriverso pushed a commit to ceriverso/select-component that referenced this pull request Jun 12, 2022
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.

4 participants