infor-design / enterprise-ng

Angular wrappers for IDS Enterprise components
Apache License 2.0
55 stars 82 forks source link

DataGrid: Groupings returns a different data when selecting a checkbox and cannot reselect it #1740

Closed jpontawe closed 6 days ago

jpontawe commented 1 month ago

Describe the bug A multi select DataGrid with groupings unfortunately returns a different data than the one selected, it is also not possible to reselect a check box   To Reproduce Steps to reproduce the behavior:

  1. Go to https://main-enterprise.demo.design.infor.com/components/datagrid/example-grouping-multiselect.html
  2. Open developer's tools and navigate to console
  3. Click the checkbox on customer id 7
  4. Inspect the Row Click -> item in the console and compare the data with the one was that was clicked
  5. Unselect customer id 7
  6. Reselect customer id 7 (cannot reselect)

Expected behavior Customer id 7's data should be printed on the browser console, and it is possible for us to reselect that record.

Version

Screenshots datagrid_checkbox_issue_demo_env

Platform

Additional context This exact bug was reproduced in our Infor app and was communicated within Microsoft Teams channel

tmcconechy commented 4 weeks ago

@jpontawe may just want to test this on 4.97.0 when released its possibly fixed

jpontawe commented 3 weeks ago

@tmcconechy what did you mean by 4.97.0? Apologies I am not familiar with this yet.

tmcconechy commented 3 weeks ago

@jpontawe

I mean try the latest version https://www.npmjs.com/package/ids-enterprise-ng/v/18.1.3 (this now is released and contains 4.97 or https://www.npmjs.com/package/ids-enterprise

And see if this is still an issue can also just test here and confirm: https://main-enterprise.demo.design.infor.com/components/datagrid/example-grouping-multiselect.html

tmcconechy commented 3 weeks ago

@jpontawe i realized that testing https://main-enterprise.demo.design.infor.com/components/datagrid/example-grouping-multiselect.html is what i need to confirm if this is fixed or not.

It is not. We will take a look when we can

jpontawe commented 3 weeks ago

@tmcconechy thank you for clarifying.

Understood, we have temporarily disabled groupings on our data grids and will wait for this to be fixed.