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

Register ajax.reloadEx() method to Datatables. #17067

Merged
merged 2 commits into from
Jul 10, 2023
Merged

Register ajax.reloadEx() method to Datatables. #17067

merged 2 commits into from
Jul 10, 2023

Conversation

maliming
Copy link
Member

@maliming maliming commented Jul 7, 2023 •
edited

Resolve #17029

@maliming maliming added this to the 7.3-final milestone Jul 10, 2023
@maliming maliming changed the base branch from rel-7.3 to dev July 10, 2023 02:08
@maliming maliming modified the milestones: 7.3-final, 7.4-preview Jul 10, 2023
@maliming maliming changed the base branch from dev to rel-7.3 July 10, 2023 02:08
@ebicoglu
Copy link
Member

@gizemmutukurt will test the functionality.
@maliming can you write the steps on how to test it

@maliming
Copy link
Member Author

@gizemmutukurt

You can create more than ten roles and set the page size as 10 and jump to the second page.

  1. create a new role again on the second page. The page should be kept on the second page after creation.
  2. delete some roles. The page will go to the first page if the count of roles is not enough on the second page.

@gizemmutukurt gizemmutukurt merged commit c80af15 into rel-7.3 Jul 10, 2023
2 checks passed
@gizemmutukurt gizemmutukurt deleted the reloadEx branch July 10, 2023 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ebicoglu ebicoglu ebicoglu approved these changes

@realLiangshiwei realLiangshiwei realLiangshiwei approved these changes

@gizemmutukurt gizemmutukurt gizemmutukurt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
7.4-preview
Development

Successfully merging this pull request may close these issues.

Consider hold current page when reloading data on datatable.
4 participants
@maliming @ebicoglu @realLiangshiwei @gizemmutukurt