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

Correctly initialize the "pageSize" prop in MessageTablePaginator #4209

Merged
merged 1 commit into from Oct 6, 2017

Conversation

Projects
None yet
2 participants
@bernd
Member

bernd commented Oct 5, 2017

Fixes an issue when no "pageSize" prop is provided. This broke in #4160.

Before this fix:

image

This needs to be merged into the 2.4 branch as well!

Correctly initialize the "pageSize" prop in MessageTablePaginator
Fixes an issue when no "pageSize" prop is provided. This broke in #4160.

@bernd bernd added this to the 2.4.0 milestone Oct 5, 2017

@dennisoelkers dennisoelkers merged commit 5a22fc2 into master Oct 6, 2017

5 checks passed

ci-web-linter Jenkins build graylog-pr-linter-check 1986 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
graylog-project/pr Jenkins build graylog-project-pr-snapshot 574 has succeeded
Details
license/cla Contributor License Agreement is signed.
Details

@dennisoelkers dennisoelkers deleted the fix-pagination branch Oct 6, 2017

dennisoelkers added a commit that referenced this pull request Oct 6, 2017

Correctly initialize the "pageSize" prop in MessageTablePaginator (#4209
)

Fixes an issue when no "pageSize" prop is provided. This broke in #4160.

(cherry picked from commit 5a22fc2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment