-
Notifications
You must be signed in to change notification settings - Fork 980
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
[WIP] Fixes #27913 - Add pagination to table component #7061
Conversation
Issues: #27913 |
d10b26c
to
1fae7aa
Compare
1 similar comment
const { REQUEST, FAILURE, SUCCESS } = createTableActionTypes(tableID); | ||
|
||
switch (action.type) { | ||
const { REQUEST, FAILURE, SUCCESS, SET_PAGINATION } = createTableActionTypes( |
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.
Looking at changes in Table, are we ok with having SET_PAGINATION for tables that are not paginated?
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.
It will just be not used, I don't see the harm in that
1fae7aa
to
5d44379
Compare
@MariaAga, this pull request is currently not mergeable. Please rebase against the develop branch and push again. If you have a remote called 'upstream' that points to this repository, you can do this by running:
This message was auto-generated by Foreman's prprocessor |
@MariaAga is this ready to test? Or do you need to rebase again? |
This is not ready at all yet |
This has been without a change for more than 3 months, therefore I will close. Feel free to reopen if you find time to work on this @MariaAga! |
Adding a pagination to the table including the needed actions and reducers.
usage example PR - #7064