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

Immutability support for array properties #2396

Closed
cagataycivici opened this issue Mar 27, 2017 · 1 comment
Closed

Immutability support for array properties #2396

cagataycivici opened this issue Mar 27, 2017 · 1 comment
Assignees
Labels
Type: Enhancement Issue contains an enhancement related to a specific component. Additional functionality has been add
Milestone

Comments

@cagataycivici
Copy link
Member

cagataycivici commented Mar 27, 2017

In case an immutable array is passed, we should not mutate the model but instead return new data. Related components are;

  • Checkbox
  • Listbox
  • AutoComplete
  • Chips
  • MultiSelect
  • SelectButton
  • DataTable (selection)
  • Tree (selection)
  • TreeTable (selection)
@cagataycivici cagataycivici added the Type: Enhancement Issue contains an enhancement related to a specific component. Additional functionality has been add label Mar 27, 2017
@cagataycivici cagataycivici added this to the 4.0.RC2 milestone Mar 27, 2017
@cagataycivici cagataycivici self-assigned this Mar 27, 2017
@cagataycivici cagataycivici changed the title Do not mutate arrays bound tp components Do not mutate arrays bound to components Mar 27, 2017
@cagataycivici cagataycivici changed the title Do not mutate arrays bound to components Support immutability for array properties Mar 28, 2017
@cagataycivici cagataycivici changed the title Support immutability for array properties Immutability support for array properties Mar 28, 2017
@born2net
Copy link

awesome, example with immutableJS please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Issue contains an enhancement related to a specific component. Additional functionality has been add
Projects
None yet
Development

No branches or pull requests

2 participants