-
Notifications
You must be signed in to change notification settings - Fork 80
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
1672 - Fix errors editing on second page #8605
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updating rows no errors encountered
@tmcconechy - are we expecting changed values to persist when going to other pages?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Still having console error
https://www.loom.com/share/78418ff18ce34bc89cc91b3debbf7b51?sid=217a87b0-5633-46c0-b166-67ec43b76bbf
@janahintal, the error in your video does not seem to be related to our library. Also, I have tested it and I don't see any errors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@n-ace-ancog no the value does not persist in server side examples. (We dont have a place to save it so when the data reloads per page its replaced) but it is saving it as to what we need, its an example issue. Since this is approved an the error from @janahintal seems to come from some other web plugin. Will merge and patch |
Explain the details for making this change. What existing problem does the pull request solve?
Fixed a bug editing on the second page in server side paging. This was caused by #7577
Related github/jira issue (required):
Fixes infor-design/enterprise-ng#1672
Steps necessary to review your pull request (required):
Included in this Pull Request: