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.

Sign up for GitHub

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

Jump to bottom

Remove ColType multiselect. Refactor Multiselect->Select isMulti #4106

Closed
minotogna opened this issue Nov 12, 2024 · 0 comments · Fixed by #4107
Closed

Remove ColType multiselect. Refactor Multiselect->Select isMulti #4106

minotogna opened this issue Nov 12, 2024 · 0 comments · Fixed by #4107
Assignees
@minotogna
Labels
enhancement New feature or request
Milestone
3.6.2

Comments

@minotogna
Copy link
Member

minotogna commented Nov 12, 2024 •
edited
Loading

  • remove colType multiSelect and add a prop to col select props: isMulti?:boolean
  • remove Cell FE MultiSelect
  • refactor Cell/Select: 1. use onChangeNodeValue 2. pass isMulti prop to Select
@minotogna minotogna self-assigned this Nov 12, 2024
@minotogna minotogna added the enhancement New feature or request label Nov 12, 2024
@minotogna minotogna added this to the 3.6.2 milestone Nov 12, 2024
@minotogna minotogna linked a pull request Nov 12, 2024 that will close this issue
4106 - Remove ColType multiselect. Refactor Multiselect->Select isMulti #4107
Merged
@mergify mergify bot closed this as completed in #4107 Nov 12, 2024
@minotogna minotogna mentioned this issue Nov 12, 2024
Open
48 tasks
@yaguzmang yaguzmang mentioned this issue Dec 18, 2024
Draft
39 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@minotogna minotogna

Labels
enhancement New feature or request
Projects
None yet
Milestone
3.6.2
Development

Successfully merging a pull request may close this issue.

4106 - Remove ColType multiselect. Refactor Multiselect->Select isMulti openforis/fra-platform
1 participant
@minotogna