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

fix(form): success feedback should display when pass hasFeedback prop and current value is valid value #41116

Merged
merged 10 commits into from
Mar 9, 2023

Conversation

kiner-tang
Copy link
Member

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • Site / documentation update
  • Demo update
  • Component style update
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Internationalization
  • Refactoring
  • Code style optimization
  • Test Case
  • Branch merge
  • Workflow
  • Other (about what?)

🔗 Related issue link

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English success feedback should display when pass hasFeedback prop and current value is valid value
🇨🇳 Chinese 当显式传递hasFeedback属性且当前值是合法值时应展示成功反馈

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2023

@kiner-tang kiner-tang marked this pull request as draft March 8, 2023 02:32
@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2023

size-limit report 📦

Path Size
./dist/antd.min.js 374.27 KB (+34 B 🔺)
./dist/antd-with-locales.min.js 431.18 KB (+30 B 🔺)

@codecov
Copy link

codecov bot commented Mar 8, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change

Comparison is base (55166d4) 100.00% compared to head (ac6b02d) 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##            master    #41116    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files          610       611     +1     
  Lines        10438     10597   +159     
  Branches      2842      2891    +49     
==========================================
+ Hits         10438     10597   +159     
Impacted Files Coverage Δ
components/form/FormItem/index.tsx 100.00% <ø> (ø)
components/form/FormItem/ItemHolder.tsx 100.00% <100.00%> (ø)
components/upload/Upload.tsx 100.00% <0.00%> (ø)
components/_util/ActionButton.tsx 100.00% <0.00%> (ø)
components/upload/utils.ts 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@kiner-tang kiner-tang marked this pull request as ready for review March 8, 2023 05:58
@kiner-tang kiner-tang requested review from zombieJ and MadCcc March 8, 2023 05:58
@kiner-tang kiner-tang requested a review from zombieJ March 9, 2023 08:27
@kiner-tang kiner-tang requested a review from zombieJ March 9, 2023 09:54
@zombieJ zombieJ merged commit c3f8424 into master Mar 9, 2023
@zombieJ zombieJ deleted the fix-formItem-feedback branch March 9, 2023 13:28
@zombieJ zombieJ mentioned this pull request Mar 13, 2023
16 tasks
RedJue pushed a commit to RedJue/ant-design that referenced this pull request Apr 4, 2023
… and current value is valid value (ant-design#41116)

* fix: success feedback should display when pass hasFeedback prop and type valid value

* feat: optimize code

* feat: update test case

* feat: optimize code

* feat: optimize code

* feat: update demo

* feat: update test case

* feat: update demo

* feat: update test case

* feat: reset demo
RedJue pushed a commit to RedJue/ant-design that referenced this pull request Apr 25, 2023
… and current value is valid value (ant-design#41116)

* fix: success feedback should display when pass hasFeedback prop and type valid value

* feat: optimize code

* feat: update test case

* feat: optimize code

* feat: optimize code

* feat: update demo

* feat: update test case

* feat: update demo

* feat: update test case

* feat: reset demo
RedJue pushed a commit to RedJue/ant-design that referenced this pull request Apr 25, 2023
… and current value is valid value (ant-design#41116)

* fix: success feedback should display when pass hasFeedback prop and type valid value

* feat: optimize code

* feat: update test case

* feat: optimize code

* feat: optimize code

* feat: update demo

* feat: update test case

* feat: update demo

* feat: update test case

* feat: reset demo
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.

Form feedback icon not showing if the input has valid value while using validateFields method
2 participants