Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(CheckList): allow using number as value #6335

Merged

Conversation

aso1datov
Copy link
Contributor

At the moment, it is allowed to use only a string as a value in the CheckList, but it is also possible to use numbers.

@github-actions
Copy link
Contributor

PR preview has been successfully built and deployed to https://antd-mobile-preview-pr-6335.surge.sh

@1587315093
Copy link
Contributor

CI fail~

@aso1datov
Copy link
Contributor Author

CI fail~

Updated

@codecov
Copy link

codecov bot commented Aug 25, 2023

Codecov Report

Patch coverage: 66.66% and project coverage change: -0.01% ⚠️

Comparison is base (8a3f426) 91.56% compared to head (1d10033) 91.56%.
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6335      +/-   ##
==========================================
- Coverage   91.56%   91.56%   -0.01%     
==========================================
  Files         306      306              
  Lines        6592     6591       -1     
  Branches     1627     1626       -1     
==========================================
- Hits         6036     6035       -1     
  Misses        521      521              
  Partials       35       35              
Files Changed Coverage Δ
src/components/check-list/check-list-item.tsx 89.28% <ø> (ø)
src/components/check-list/context.tsx 100.00% <ø> (ø)
src/components/check-list/index.tsx 100.00% <ø> (ø)
src/components/check-list/check-list.tsx 90.47% <50.00%> (ø)
src/components/cascader-view/cascader-view.tsx 94.36% <100.00%> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@1587315093 1587315093 merged commit adff6a1 into ant-design:master Aug 31, 2023
8 checks passed
@aso1datov aso1datov deleted the aso1datov/check-list-value-type branch September 1, 2023 12:02
cytle pushed a commit to cytle/ant-design-mobile that referenced this pull request Sep 11, 2023
* feat(CheckList): allow using number as value

* refactor(CheckList): move value type to component

* fix(CascaderView): update CascaderValue type

* refactor(CheckList): update type import path
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.

None yet

2 participants