-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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: add dynamic list input #6146
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6146 +/- ##
============================================
- Coverage 56.91% 55.04% -1.88%
- Complexity 3319 3490 +171
============================================
Files 587 635 +48
Lines 18968 21388 +2420
Branches 1401 1496 +95
============================================
+ Hits 10795 11772 +977
- Misses 7594 9010 +1416
- Partials 579 606 +27 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Ryan Wang <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: JohnNiang The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind feature
/area ui
/milestone 2.17.x
What this PR does / why we need it:
为 formkit 增加动态列表的 input。
使用方式:
Note
list
组件有且只有一个子节点,并且必须为子节点传递index
属性。若想提供多个字段,则建议使用group
组件包裹。How to test it?
测试动态数组是否正常可用。保存的结果是否为
{users: ["", ""]}
Which issue(s) this PR fixes:
Fixes #6098
Does this PR introduce a user-facing change?