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

Datagrid: Server Paging - Cell editor is only working on First page but not Other pages. #1672

Closed
vsatyamsetty opened this issue Apr 5, 2024 · 5 comments · Fixed by infor-design/enterprise#8605
Labels
priority: high type: bug 🐛 [2] Velocity rating (Fibonacci)

Comments

@vsatyamsetty
Copy link

vsatyamsetty commented Apr 5, 2024

Describe the bug
Datagrid - Paging on server side is working but editing/modifying a cell is not updating the value on the 2nd page.
Only modifying on the cell is working on 1st page, unable to modify the cell on any other pages.

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://main-enterprise.demo.design.infor.com/components/datagrid/test-expandable-paging-server-side.html
  2. Click on page 2.
  3. Open the console tab in the browser.
  4. Modify any of the value in the 'Product Name' Column.
  5. See the error in the console. Also, the value is being updated empty in the ProductName column.

Expected behavior
The value should be updated by modifying the cell(ProductName) as expected on every page. Not just in the 1st page.

Version

  • ids-enterprise-ng: [ v4.93.0]

Screenshots
image

Platform

  • Browser Name [e.g. chrome, safari, stock browser]

Additional context
Expecting this fix ASAP.

@vsatyamsetty vsatyamsetty changed the title Server Paging in Datagrid - Cell editor is only working on First page but not Other pages. Datagrid: Server Paging - Cell editor is only working on First page but not Other pages. Apr 5, 2024
@tmcconechy tmcconechy added type: bug 🐛 [2] Velocity rating (Fibonacci) priority: high labels Apr 5, 2024
@tmcconechy
Copy link
Member

@ericangeles one more issue for 4.84

@vsatyamsetty
Copy link
Author

vsatyamsetty commented Apr 10, 2024

Hi @tmcconechy ,
Is it completed?
Will this work for angular?
Which package version I need to define in the package.JSON file?
"ids-enterprise-ng": "" - what would be the version here?

Thanks,
Phani.

@tmcconechy
Copy link
Member

@vsatyamsetty its fixed in 4.94.0 -> ids-enterprise-ng@17.6.1

@vsatyamsetty
Copy link
Author

vsatyamsetty commented Apr 11, 2024

@tmcconechy ,
Thanks.

The header is not blue color. The header is white color.
Is this the expected behavior for this latest version 4.94.0?

Version ids-enterprise-ng@17.6.1

image

The below screenshot is from the application that I am using.
Version ids-enterprise-ng@16.7.1

image

@tmcconechy
Copy link
Member

Yes the header is now white (alabaster). It can be set to blue with personalization but thats no longer required by design.

https://main-enterprise.demo.design.infor.com/components/header/example-index.html
VS
https://main-enterprise.demo.design.infor.com/components/header/example-index.html?colors=0066D4

So this is a correct default now @vsatyamsetty

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: high type: bug 🐛 [2] Velocity rating (Fibonacci)
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

2 participants