Skip to content

Conversation

banqinghe
Copy link
Contributor

Hi,我最近在工作中 fork 并使用了这个项目,感谢作者的维护工作。今天我也遇到了 #21 中提到的必选项无法设置的问题,自己修复了一下。

原本的代码里使用 parentSchemaDepth 属性来追踪当前层级的方式个人感觉可以直接使用 namePath + getValueByPath 的方式来代替,代码的可读性也更强一些(调代码的时候这个 parentSchemaDepth 行为确实让我有些困惑)。这个 PR 更改了 updateRequiredProperty 的逻辑,也解决了上文中提到的 issue 所描述的问题。

@lin-mt lin-mt merged commit cf75732 into lin-mt:main Mar 18, 2025
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.

2 participants