fix(Transfer): fix transfer issue when trying to select item from the…

andreyk1512 commented 2 weeks ago

… right column while it has "unknown-key" within targetKeys

中文版模板 / Chinese template

🤔 This is a ...

[ ] New feature
[x] 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

fix #48663

💡 Background and solution

Transfer component crashes the app when trying to select item from the right column while it has "unknown-key" within targetKeys.

To solve this I've added optional chaining when check 'disabled' property in 'item' object. It should help to filter empty items and avoid the issue.

📝 Changelog

Language	Changelog
🇺🇸 English	Fix Transfer component issue when trying to select item from the right column while it has "unknown-key" within targetKeys
🇨🇳 Chinese

☑️ Self-Check before Merge

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

[x] Doc is updated/provided or not needed
[x] Demo is updated/provided or not needed
[x] TypeScript definition is updated/provided or not needed
[x] Changelog is provided or not needed

stackblitz[bot] commented 2 weeks ago

Run & review this pull request in StackBlitz Codeflow.

github-actions[bot] commented 2 weeks ago

👁 Visual Regression Report for PR #48664 Passed ✅

🎯 Target branch: master (c8bd145ab4ca0c5b234be327b7f793137cf293d7) 📖 View Full Report ↗︎

🎊 Congrats! No visual-regression diff found.

codesandbox-ci[bot] commented 2 weeks ago

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

github-actions[bot] commented 2 weeks ago

🎉 Thank you for your contribution! If you have not yet joined our DingTalk community group, please feel free to join us (when joining, please provide the link to this PR).

🎉 感谢您的贡献！如果您还没有加入钉钉社区群，请扫描下方二维码加入我们（加群时请提供此 PR 链接）。

Wxh16144 commented 2 weeks ago

The changelog ci check failed, and the test case is missing. 😅

ant-design / ant-design