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

Changed pagination of organizer page #385

Merged
merged 2 commits into from Jan 20, 2017
Merged

Changed pagination of organizer page #385

merged 2 commits into from Jan 20, 2017

Conversation

jfwiebe
Copy link
Contributor

@jfwiebe jfwiebe commented Jan 19, 2017

...to show more than one event per page. Maybe this value should become a configurable value in the future.

image

...to show more than one event per page. Maybe this value should become a configurable value in the future.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.522% when pulling 18a4826 on jfwiebe:patch-1 into 7300c60 on pretix:master.

@raphaelm
Copy link
Member

Oops, that 1 is probably a debugging artifact. I think even 30 would be good, as the rows are very narrow.

@jfwiebe
Copy link
Contributor Author

jfwiebe commented Jan 20, 2017

I agree with you, thats right.

@raphaelm
Copy link
Member

Awesome, thank you! :)

@raphaelm raphaelm merged commit 520b978 into pretix:master Jan 20, 2017
@jfwiebe jfwiebe deleted the patch-1 branch January 20, 2017 14:07
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 79.491% when pulling e29f3dd on jfwiebe:patch-1 into 7300c60 on pretix:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants