Skip to content

fix: 优化树结构实体的保存 #319

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

Merged
merged 23 commits into from
Apr 28, 2025
Merged

fix: 优化树结构实体的保存 #319

merged 23 commits into from
Apr 28, 2025

Conversation

kyouji
Copy link
Contributor

@kyouji kyouji commented Apr 28, 2025

未传入parentId时,不更新path

@CLAassistant
Copy link

CLAassistant commented Apr 28, 2025

CLA assistant check
All committers have signed the CLA.

@zhou-hao
Copy link
Member

现在将树节点设置为顶级节点时 parentId设置的是啥?

@kyouji
Copy link
Contributor Author

kyouji commented Apr 28, 2025

现在将树节点设置为顶级节点时 parentId设置的是啥?

设置的空字符串,然后再订阅实体事件改成null

@zhou-hao
Copy link
Member

这样修改之后还能设置为顶级节点吧?

@kyouji
Copy link
Contributor Author

kyouji commented Apr 28, 2025

这样修改之后还能设置为顶级节点吧?

可以再设置为root节点。这是通用的规则吧,PUT不传字段(null)就表示不修改它,传空字符串表示清空。

之前在组织中订阅实体事件改成null,是为了适配页面组件的反显

@zhou-hao zhou-hao merged commit b335252 into hs-web:master Apr 28, 2025
1 check passed
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.

3 participants