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: 修复 quickEdit inline 模式 name 为带层级时发送的数据不符合预期 Close: #8756 #8759

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

2betop
Copy link
Collaborator

@2betop 2betop commented Nov 15, 2023

What

🤖 Generated by Copilot at 92a652c

This pull request improves the handling of nested data variables in QuickEdit components. It uses an enhanced setVariable function from amis-core that can deal with numeric keys and arrays.

🤖 Generated by Copilot at 92a652c

We're setting variables with a stack and a key
We're creating arrays when we need them, aye
We're using setVariable from amis-core
We're heaving together on the count of three

Why

Close: #8756

How

🤖 Generated by Copilot at 92a652c

  • Improve the logic of setting a variable with a nested key in amis-core (link, link)
  • Use the setVariable function from amis-core in the QuickEdit component in amis (link, link)

@github-actions github-actions bot added the fix label Nov 15, 2023
Copy link

👍 Thanks for this!
🏷 I have applied any labels matching special text in your issue.

Please review the labels and make any necessary changes.

@hsm-lv hsm-lv merged commit ee6fc42 into baidu:master Nov 17, 2023
5 checks passed
@lengqingfeng
Copy link
Contributor

截屏2024-01-15 15 47 10 你这么改导致列字段不唯一,引起组件不受控。这个改怎么改。@2betop @nwind 吴多益

@2betop 2betop deleted the fix-quickEdit-name-path branch January 17, 2024 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants